[collectd] [PATCH] mysql plugin: add support for multipledatabases

Doug MacEachern Doug.MacEachern at hyperic.com
Tue Mar 17 16:20:19 CET 2009


Hi Sebastian,

...
Thanks for the feedback, looks like Florian has taken care of the other
comments :)
...

> > +static MYSQL *getconnection (mysql_database_t *db)
> >  {
> >  	static MYSQL *con;
> >  	static int    state;
> 
> Uhm ... that's not going to work. The connection and state information
> have to be stored in the mysql_database_t object as well.

Whoops!  I completely missed that, patch attached.

Thanks,
-Doug
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-un-static-ize-mysql-connections.patch
Type: application/octet-stream
Size: 3463 bytes
Desc: 0001-un-static-ize-mysql-connections.patch
Url : http://mailman.verplant.org/pipermail/collectd/attachments/20090317/2cb671c7/attachment.obj 


More information about the collectd mailing list