maria-developers team mailing list archive
Mailing list archive
Re: WL#177 add server-side error logging
On Jul 16, Vladislav Vaintroub wrote:
> > > +MYSQL_ADD_PLUGIN(sql_errlog sql_errlog.c
> > > + MODULE_ONLY MODULE_OUTPUT_NAME "sql_erlg")
> > why non-default module output name?
> > I think MODULE_OUTPUT_NAME should only be used when converting old
> > plugins
> > that have to keep non-default module name for compatibility reasons.
> Correct. I was wondering about why MODULE_ONLY too in this case. Is
> it bad to optionally link this into the server statically?
> > new plugins don't need to use MODULE_OUTPUT_NAME at all.
> MYSQL_ADD_PLUGIN(sql_errlog sql_errlog.c) should normally be just
It's very good that in MYSQL_ADD_PLUGIN macro you tried to pick
reasonable defaults and to keep amount of boilerplate code to a minimum.