mysql plugin: Added support for master/slave statistics.
[collectd.git] / src / mysql.c
index 9b874f1..2314f23 100644 (file)
@@ -1,6 +1,7 @@
 /**
  * collectd - src/mysql.c
- * Copyright (C) 2006,2007  Florian octo Forster
+ * Copyright (C) 2006-2009  Florian octo Forster
+ * Copyright (C) 2009  Sebastian tokkee Harl
  *
  * This program is free software; you can redistribute it and/or modify it
  * under the terms of the GNU General Public License as published by the
@@ -17,6 +18,7 @@
  *
  * Authors:
  *   Florian octo Forster <octo at verplant.org>
+ *   Sebastian tokkee Harl <sh at tokkee.org>
  **/
 
 #include "collectd.h"
 
 struct mysql_database_s /* {{{ */
 {
+       /* instance == NULL  =>  legacy mode */
+       char *instance;
        char *host;
        char *user;
        char *pass;
-       char *name;
+       char *database;
        char *socket;
        int   port;
+
+       int   master_stats;
+       int   slave_stats;
+
+       MYSQL *con;
+       int    state;
 };
 typedef struct mysql_database_s mysql_database_t; /* }}} */
 
-static mysql_database_t **databases     = NULL;
-static size_t             databases_num = 0;
+static int mysql_read (user_data_t *ud);
 
-static void mysql_database_free (mysql_database_t *db) /* {{{ */
+static void mysql_database_free (void *arg) /* {{{ */
 {
+       mysql_database_t *db;
+
+       DEBUG ("mysql plugin: mysql_database_free (arg = %p);", arg);
+
+       db = (mysql_database_t *) arg;
+
        if (db == NULL)
                return;
 
+       if (db->con != NULL)
+               mysql_close (db->con);
+
        sfree (db->host);
        sfree (db->user);
        sfree (db->pass);
        sfree (db->socket);
+       sfree (db->instance);
+       sfree (db->database);
        sfree (db);
 } /* }}} void mysql_database_free */
 
@@ -112,10 +132,45 @@ static int mysql_config_set_int (int *ret_int, /* {{{ */
        return (0);
 } /* }}} int mysql_config_set_int */
 
-static int mysql_config_add_database (oconfig_item_t *ci) /* {{{ */
+static int mysql_config_set_boolean (int *ret_boolean, /* {{{ */
+                               oconfig_item_t *ci)
+{
+       int status = 0;
+
+       if (ci->values_num != 1)
+               status = -1;
+
+       if (status == 0)
+       {
+               if (ci->values[0].type == OCONFIG_TYPE_BOOLEAN)
+                       *ret_boolean = ci->values[0].value.boolean;
+               else if (ci->values[0].type == OCONFIG_TYPE_STRING)
+               {
+                       if (IS_TRUE (ci->values[0].value.string))
+                               *ret_boolean = 1;
+                       else if (IS_FALSE (ci->values[0].value.string))
+                               *ret_boolean = 0;
+                       else
+                               status = -1;
+               }
+               else
+                       status = -1;
+       }
+
+       if (status != 0)
+       {
+               WARNING ("mysql plugin: The `%s' config option "
+                       "needs exactly one boolean argument.", ci->key);
+               return (-1);
+       }
+       return (0);
+} /* }}} mysql_config_set_boolean */
+
+static int mysql_config (oconfig_item_t *ci) /* {{{ */
 {
        mysql_database_t *db;
-       int status;
+       int plugin_block;
+       int status = 0;
        int i;
 
        if ((ci->values_num != 1)
@@ -134,11 +189,36 @@ static int mysql_config_add_database (oconfig_item_t *ci) /* {{{ */
        }
        memset (db, 0, sizeof (*db));
 
-       status = mysql_config_set_string (&db->name, ci);
-       if (status != 0)
+       /* initialize all the pointers */
+       db->host     = NULL;
+       db->user     = NULL;
+       db->pass     = NULL;
+       db->database = NULL;
+       db->socket   = NULL;
+       db->con      = NULL;
+
+       plugin_block = 1;
+       if (strcasecmp ("Plugin", ci->key) == 0)
+       {
+               db->instance = NULL;
+       }
+       else if (strcasecmp ("Database", ci->key) == 0)
+       {
+               plugin_block = 0;
+               status = mysql_config_set_string (&db->instance, ci);
+               if (status != 0)
+               {
+                       sfree (db);
+                       return (status);
+               }
+               assert (db->instance != NULL);
+               db->database = strdup (db->instance);
+       }
+       else
        {
-               sfree (db);
-               return (status);
+               ERROR ("mysql plugin: mysql_config: "
+                               "Invalid key: %s", ci->key);
+               return (-1);
        }
 
        /* Fill the `mysql_database_t' structure.. */
@@ -156,6 +236,25 @@ static int mysql_config_add_database (oconfig_item_t *ci) /* {{{ */
                        status = mysql_config_set_int (&db->port, child);
                else if (strcasecmp ("Socket", child->key) == 0)
                        status = mysql_config_set_string (&db->socket, child);
+               /* Check if we're currently handling the `Plugin' block. If so,
+                * handle `Database' _blocks_, too. */
+               else if ((plugin_block != 0)
+                               && (strcasecmp ("Database", child->key) == 0)
+                               && (child->children != NULL))
+               {
+                       /* If `plugin_block > 1', there has been at least one
+                        * `Database' block */
+                       plugin_block++;
+                       status = mysql_config (child);
+               }
+               /* Now handle ordinary `Database' options (without children) */
+               else if ((strcasecmp ("Database", child->key) == 0)
+                               && (child->children == NULL))
+                       status = mysql_config_set_string (&db->database, child);
+               else if (strcasecmp ("MasterStats", child->key) == 0)
+                       status = mysql_config_set_boolean (&db->master_stats, child);
+               else if (strcasecmp ("SlaveStats", child->key) == 0)
+                       status = mysql_config_set_boolean (&db->slave_stats, child);
                else
                {
                        WARNING ("mysql plugin: Option `%s' not allowed here.", child->key);
@@ -166,198 +265,194 @@ static int mysql_config_add_database (oconfig_item_t *ci) /* {{{ */
                        break;
        }
 
+       /* Check if there were any `Database' blocks. */
+       if (plugin_block > 1)
+       {
+               /* There were connection blocks. Don't use any legacy stuff. */
+               if ((db->host != NULL)
+                       || (db->user != NULL)
+                       || (db->pass != NULL)
+                       || (db->database != NULL)
+                       || (db->socket != NULL)
+                       || (db->port != 0))
+               {
+                       WARNING ("mysql plugin: At least one <Database> "
+                                       "block has been found. The legacy "
+                                       "configuration will be ignored.");
+               }
+               mysql_database_free (db);
+               return (0);
+       }
+       else if (plugin_block != 0)
+       {
+               WARNING ("mysql plugin: You're using the legacy "
+                               "configuration options. Please consider "
+                               "updating your configuration!");
+       }
+
        /* Check that all necessary options have been given. */
        while (status == 0)
        {
-               if ((db->port < 0) || (db->port >= 65535))
+               /* Zero is allowed and automatically handled by
+                * `mysql_real_connect'. */
+               if ((db->port < 0) || (db->port > 65535))
+               {
+                       ERROR ("mysql plugin: Database %s: Port number out "
+                                       "of range: %i",
+                                       (db->instance != NULL)
+                                       ? db->instance
+                                       : "<legacy>",
+                                       db->port);
+                       status = -1;
+               }
+               if (db->database == NULL)
                {
-                       ERROR ("mysql plugin: Port number out of range: %i",
-                              db->port);
+                       ERROR ("mysql plugin: No `Database' configured");
                        status = -1;
                }
                break;
        } /* while (status == 0) */
 
-       /* If all went well, add this database to the global list of databases. */
+       /* If all went well, register this database for reading */
        if (status == 0)
        {
-               mysql_database_t **temp;
+               user_data_t ud;
+               char cb_name[DATA_MAX_NAME_LEN];
 
-               temp = (mysql_database_t **) realloc (databases,
-                                                    sizeof (*databases) * (databases_num + 1));
-               if (temp == NULL)
-               {
-                       ERROR ("mysql plugin: realloc failed");
-                       status = -1;
-               }
+               DEBUG ("mysql plugin: Registering new read callback: %s", db->database);
+
+               memset (&ud, 0, sizeof (ud));
+               ud.data = (void *) db;
+               ud.free_func = mysql_database_free;
+
+               if (db->database != NULL)
+                       ssnprintf (cb_name, sizeof (cb_name), "mysql-%s",
+                                       db->database);
                else
-               {
-                       databases = temp;
-                       databases[databases_num] = db;
-                       databases_num++;
-               }
-       }
+                       sstrncpy (cb_name, "mysql", sizeof (cb_name));
 
-       if (status != 0)
+               plugin_register_complex_read (cb_name, mysql_read,
+                                             /* interval = */ NULL, &ud);
+       }
+       else
        {
                mysql_database_free (db);
                return (-1);
        }
 
        return (0);
-} /* }}} int mysql_config_add_database */
-
-static int mysql_config (oconfig_item_t *ci) /* {{{ */
-{
-       int status = 0;
-       int i;
-       oconfig_item_t *lci = NULL; /* legacy config */
-
-       for (i = 0; i < ci->children_num; i++)
-       {
-               oconfig_item_t *child = ci->children + i;
-
-               if (strcasecmp ("Database", child->key) == 0 && child->children_num > 0)
-                       mysql_config_add_database (child);
-               else
-               {
-                       /* legacy mode - convert to <Database ...> config */
-                       if (lci == NULL)
-                       {
-                               lci = malloc (sizeof(*lci));
-                               if (lci == NULL)
-                               {
-                                       ERROR ("mysql plugin: malloc failed.");
-                                       return (-1);
-                               }
-                               memset (lci, '\0', sizeof (*lci));
-                       }
-                       if (strcasecmp ("Database", child->key) == 0)
-                       {
-                               lci->key = child->key;
-                               lci->values = child->values;
-                               lci->values_num = child->values_num;
-                               lci->parent = child->parent;
-                       }
-                       else
-                       {
-                               lci->children_num++;
-                               lci->children =
-                                       realloc (lci->children,
-                                                lci->children_num * sizeof (*child));
-                               if (lci->children == NULL)
-                               {
-                                       ERROR ("mysql plugin: realloc failed.");
-                                       return (-1);
-                               }
-                               memcpy (&lci->children[lci->children_num-1], child, sizeof (*child));
-                       }
-               }
-       } /* for (ci->children) */
-
-       if (lci)
-       {
-               if (lci->key == NULL)
-               {
-                       ERROR ("mysql plugin: no Database configured.");
-                       status = -1;
-               }
-               else
-                       mysql_config_add_database (lci);
-               sfree (lci->children);
-               sfree (lci);
-       }
-       return (status);
 } /* }}} int mysql_config */
 
+/* }}} End of configuration handling functions */
+
 static MYSQL *getconnection (mysql_database_t *db)
 {
-       static MYSQL *con;
-       static int    state;
-
-       static int wait_for = 0;
-       static int wait_increase = 60;
-
-       if (state != 0)
+       if (db->state != 0)
        {
                int err;
-               if ((err = mysql_ping (con)) != 0)
+               if ((err = mysql_ping (db->con)) != 0)
                {
-                       WARNING ("mysql_ping failed: %s", mysql_error (con));
-                       state = 0;
+                       WARNING ("mysql_ping failed: %s", mysql_error (db->con));
+                       db->state = 0;
                }
                else
                {
-                       state = 1;
-                       return (con);
+                       db->state = 1;
+                       return (db->con);
                }
        }
 
-       if (wait_for > 0)
+       if ((db->con = mysql_init (db->con)) == NULL)
        {
-               wait_for -= interval_g;
+               ERROR ("mysql_init failed: %s", mysql_error (db->con));
+               db->state = 0;
                return (NULL);
        }
 
-       wait_for = wait_increase;
-       wait_increase *= 2;
-       if (wait_increase > 86400)
-               wait_increase = 86400;
-
-       if ((con = mysql_init (con)) == NULL)
+       if (mysql_real_connect (db->con, db->host, db->user, db->pass,
+                               db->database, db->port, db->socket, 0) == NULL)
        {
-               ERROR ("mysql_init failed: %s", mysql_error (con));
-               state = 0;
+               ERROR ("mysql_real_connect failed: %s", mysql_error (db->con));
+               db->state = 0;
                return (NULL);
        }
-
-       if (mysql_real_connect (con, db->host, db->user, db->pass, db->name, db->port, db->socket, 0) == NULL)
+       else
        {
-               ERROR ("mysql_real_connect failed: %s", mysql_error (con));
-               state = 0;
-               return (NULL);
+               db->state = 1;
+               return (db->con);
        }
+} /* static MYSQL *getconnection (mysql_database_t *db) */
+
+static void set_host (mysql_database_t *db, value_list_t *vl)
+{
+       /* XXX legacy mode - use hostname_g */
+       if (db->instance == NULL)
+               sstrncpy (vl->host, hostname_g, sizeof (vl->host));
        else
        {
-               state = 1;
-               wait_for = 0;
-               wait_increase = 60;
-               return (con);
+               if ((db->host == NULL)
+                               || (strcmp ("", db->host) == 0)
+                               || (strcmp ("localhost", db->host) == 0))
+                       sstrncpy (vl->host, hostname_g, sizeof (vl->host));
+               else
+                       sstrncpy (vl->host, db->host, sizeof (vl->host));
        }
-} /* static MYSQL *getconnection (void) */
+}
 
 static void set_plugin_instance (mysql_database_t *db, value_list_t *vl)
 {
        /* XXX legacy mode - no plugin_instance */
-       if (databases_num > 0)
-               sstrncpy (vl->plugin_instance, db->name, sizeof (vl->plugin_instance));
+       if (db->instance == NULL)
+               sstrncpy (vl->plugin_instance, "",
+                               sizeof (vl->plugin_instance));
+       else
+               sstrncpy (vl->plugin_instance, db->instance,
+                               sizeof (vl->plugin_instance));
 }
 
-static void counter_submit (const char *type, const char *type_instance,
-               counter_t value, mysql_database_t *db)
+static void submit (const char *type, const char *type_instance,
+               value_t *values, size_t values_len, mysql_database_t *db)
 {
-       value_t values[1];
        value_list_t vl = VALUE_LIST_INIT;
 
-       values[0].counter = value;
+       vl.values     = values;
+       vl.values_len = values_len;
+
+       set_host (db, &vl);
 
-       vl.values = values;
-       vl.values_len = 1;
-       sstrncpy (vl.host, hostname_g, sizeof (vl.host));
        sstrncpy (vl.plugin, "mysql", sizeof (vl.plugin));
-       sstrncpy (vl.type, type, sizeof (vl.type));
-       sstrncpy (vl.type_instance, type_instance, sizeof (vl.type_instance));
        set_plugin_instance (db, &vl);
 
+       sstrncpy (vl.type, type, sizeof (vl.type));
+       if (type_instance != NULL)
+               sstrncpy (vl.type_instance, type_instance, sizeof (vl.type_instance));
+
        plugin_dispatch_values (&vl);
+} /* submit */
+
+static void counter_submit (const char *type, const char *type_instance,
+               counter_t value, mysql_database_t *db)
+{
+       value_t values[1];
+
+       values[0].counter = value;
+       submit (type, type_instance, values, STATIC_ARRAY_SIZE (values), db);
 } /* void counter_submit */
 
+static void gauge_submit (const char *type, const char *type_instance,
+               gauge_t value, mysql_database_t *db)
+{
+       value_t values[1];
+
+       values[0].gauge = value;
+       submit (type, type_instance, values, STATIC_ARRAY_SIZE (values), db);
+} /* void gauge_submit */
+
 static void qcache_submit (counter_t hits, counter_t inserts,
                counter_t not_cached, counter_t lowmem_prunes,
                gauge_t queries_in_cache, mysql_database_t *db)
 {
        value_t values[5];
-       value_list_t vl = VALUE_LIST_INIT;
 
        values[0].counter = hits;
        values[1].counter = inserts;
@@ -365,62 +460,170 @@ static void qcache_submit (counter_t hits, counter_t inserts,
        values[3].counter = lowmem_prunes;
        values[4].gauge   = queries_in_cache;
 
-       vl.values = values;
-       vl.values_len = 5;
-       sstrncpy (vl.host, hostname_g, sizeof (vl.host));
-       sstrncpy (vl.plugin, "mysql", sizeof (vl.plugin));
-       sstrncpy (vl.type, "mysql_qcache", sizeof (vl.type));
-       set_plugin_instance (db, &vl);
-
-       plugin_dispatch_values (&vl);
+       submit ("mysql_qcache", NULL, values, STATIC_ARRAY_SIZE (values), db);
 } /* void qcache_submit */
 
 static void threads_submit (gauge_t running, gauge_t connected, gauge_t cached,
                counter_t created, mysql_database_t *db)
 {
        value_t values[4];
-       value_list_t vl = VALUE_LIST_INIT;
 
        values[0].gauge   = running;
        values[1].gauge   = connected;
        values[2].gauge   = cached;
        values[3].counter = created;
 
-       vl.values = values;
-       vl.values_len = 4;
-       sstrncpy (vl.host, hostname_g, sizeof (vl.host));
-       sstrncpy (vl.plugin, "mysql", sizeof (vl.plugin));
-       sstrncpy (vl.type, "mysql_threads", sizeof (vl.type));
-       set_plugin_instance (db, &vl);
-
-       plugin_dispatch_values (&vl);
+       submit ("mysql_threads", NULL, values, STATIC_ARRAY_SIZE (values), db);
 } /* void threads_submit */
 
 static void traffic_submit (counter_t rx, counter_t tx, mysql_database_t *db)
 {
        value_t values[2];
-       value_list_t vl = VALUE_LIST_INIT;
 
        values[0].counter = rx;
        values[1].counter = tx;
 
-       vl.values = values;
-       vl.values_len = 2;
-       sstrncpy (vl.host, hostname_g, sizeof (vl.host));
-       sstrncpy (vl.plugin, "mysql", sizeof (vl.plugin));
-       sstrncpy (vl.type, "mysql_octets", sizeof (vl.type));
-       set_plugin_instance (db, &vl);
-
-       plugin_dispatch_values (&vl);
+       submit ("mysql_octets", NULL, values, STATIC_ARRAY_SIZE (values), db);
 } /* void traffic_submit */
 
-static int mysql_read_database (mysql_database_t *db)
+static MYSQL_RES *exec_query (MYSQL *con, const char *query)
+{
+       MYSQL_RES *res;
+
+       int query_len = strlen (query);
+
+       if (mysql_real_query (con, query, query_len))
+       {
+               ERROR ("mysql plugin: Failed to execute query: %s",
+                               mysql_error (con));
+               INFO ("mysql plugin: SQL query was: %s", query);
+               return (NULL);
+       }
+
+       res = mysql_store_result (con);
+       if (res == NULL)
+       {
+               ERROR ("mysql plugin: Failed to store query result: %s",
+                               mysql_error (con));
+               INFO ("mysql plugin: SQL query was: %s", query);
+               return (NULL);
+       }
+
+       return (res);
+} /* exec_query */
+
+static int mysql_read_master_stats (mysql_database_t *db, MYSQL *con)
 {
+       MYSQL_RES *res;
+       MYSQL_ROW  row;
+
+       char *query;
+       int   field_num;
+       unsigned long long position;
+
+       query = "SHOW MASTER STATUS";
+
+       res = exec_query (con, query);
+       if (res == NULL)
+               return (-1);
+
+       row = mysql_fetch_row (res);
+       if (row == NULL)
+       {
+               ERROR ("mysql plugin: Failed to get master statistics: "
+                               "`%s' did not return any rows.", query);
+               return (-1);
+       }
+
+       field_num = mysql_num_fields (res);
+       if (field_num < 2)
+       {
+               ERROR ("mysql plugin: Failed to get master statistics: "
+                               "`%s' returned less than two columns.", query);
+               return (-1);
+       }
+
+       position = atoll (row[1]);
+       counter_submit ("mysql_log_position", "master-bin", position, db);
+
+       row = mysql_fetch_row (res);
+       if (row != NULL)
+               WARNING ("mysql plugin: `%s' returned more than one row - "
+                               "ignoring further results.", query);
+
+       mysql_free_result (res);
+
+       return (0);
+} /* mysql_read_master_stats */
+
+static int mysql_read_slave_stats (mysql_database_t *db, MYSQL *con)
+{
+       MYSQL_RES *res;
+       MYSQL_ROW  row;
+
+       char *query;
+       int   field_num;
+
+       /* WTF? libmysqlclient does not seem to provide any means to
+        * translate a column name to a column index ... :-/ */
+       const int READ_MASTER_LOG_POS_IDX   = 6;
+       const int EXEC_MASTER_LOG_POS_IDX   = 21;
+       const int SECONDS_BEHIND_MASTER_IDX = 32;
+
+       unsigned long long counter;
+       double gauge;
+
+       query = "SHOW SLAVE STATUS";
+
+       res = exec_query (con, query);
+       if (res == NULL)
+               return (-1);
+
+       row = mysql_fetch_row (res);
+       if (row == NULL)
+       {
+               ERROR ("mysql plugin: Failed to get slave statistics: "
+                               "`%s' did not return any rows.", query);
+               return (-1);
+       }
+
+       field_num = mysql_num_fields (res);
+       if (field_num < 33)
+       {
+               ERROR ("mysql plugin: Failed to get slave statistics: "
+                               "`%s' returned less than 33 columns.", query);
+               return (-1);
+       }
+
+       counter = atoll (row[READ_MASTER_LOG_POS_IDX]);
+       counter_submit ("mysql_log_position", "slave-read", counter, db);
+
+       counter = atoll (row[EXEC_MASTER_LOG_POS_IDX]);
+       counter_submit ("mysql_log_position", "slave-exec", counter, db);
+
+       if (row[SECONDS_BEHIND_MASTER_IDX] != NULL)
+       {
+               gauge = atof (row[SECONDS_BEHIND_MASTER_IDX]);
+               gauge_submit ("time_offset", NULL, gauge, db);
+       }
+
+       row = mysql_fetch_row (res);
+       if (row != NULL)
+               WARNING ("mysql plugin: `%s' returned more than one row - "
+                               "ignoring further results.", query);
+
+       mysql_free_result (res);
+
+       return (0);
+} /* mysql_read_slave_stats */
+
+static int mysql_read (user_data_t *ud)
+{
+       mysql_database_t *db;
        MYSQL     *con;
        MYSQL_RES *res;
        MYSQL_ROW  row;
        char      *query;
-       int        query_len;
        int        field_num;
 
        unsigned long long qcache_hits          = 0ULL;
@@ -437,6 +640,14 @@ static int mysql_read_database (mysql_database_t *db)
        unsigned long long traffic_incoming = 0ULL;
        unsigned long long traffic_outgoing = 0ULL;
 
+       if ((ud == NULL) || (ud->data == NULL))
+       {
+               ERROR ("mysql plugin: mysql_database_read: Invalid user data.");
+               return (-1);
+       }
+
+       db = (mysql_database_t *) ud->data;
+
        /* An error message will have been printed in this case */
        if ((con = getconnection (db)) == NULL)
                return (-1);
@@ -445,21 +656,9 @@ static int mysql_read_database (mysql_database_t *db)
        if (mysql_get_server_version (con) >= 50002)
                query = "SHOW GLOBAL STATUS";
 
-       query_len = strlen (query);
-
-       if (mysql_real_query (con, query, query_len))
-       {
-               ERROR ("mysql_real_query failed: %s\n",
-                               mysql_error (con));
+       res = exec_query (con, query);
+       if (res == NULL)
                return (-1);
-       }
-
-       if ((res = mysql_store_result (con)) == NULL)
-       {
-               ERROR ("mysql_store_result failed: %s\n",
-                               mysql_error (con));
-               return (-1);
-       }
 
        field_num = mysql_num_fields (res);
        while ((row = mysql_fetch_row (res)))
@@ -533,30 +732,11 @@ static int mysql_read_database (mysql_database_t *db)
 
        traffic_submit  (traffic_incoming, traffic_outgoing, db);
 
-       /* mysql_close (con); */
-
-       return (0);
-} /* int mysql_read_database */
-
-static int mysql_read (void)
-{
-       size_t i;
-       int success = 0;
-       int status;
+       if (db->master_stats)
+               mysql_read_master_stats (db, con);
 
-       for (i = 0; i < databases_num; i++)
-       {
-               status = mysql_read_database (databases[i]);
-               if (status == 0)
-                       success++;
-       }
-
-       if (success == 0)
-       {
-               ERROR ("mysql plugin: No database could be read. Will return an error so "
-                      "the plugin will be delayed.");
-               return (-1);
-       }
+       if (db->slave_stats)
+               mysql_read_slave_stats (db, con);
 
        return (0);
 } /* int mysql_read */
@@ -564,5 +744,4 @@ static int mysql_read (void)
 void module_register (void)
 {
        plugin_register_complex_config ("mysql", mysql_config);
-       plugin_register_read ("mysql", mysql_read);
 } /* void module_register */