Code

write_redis: re-add colon dropped in b7984797
authorMarc Fournier <marc.fournier@camptocamp.com>
Sat, 8 Nov 2014 23:16:11 +0000 (00:16 +0100)
committerMarc Fournier <marc.fournier@camptocamp.com>
Sat, 8 Nov 2014 23:16:11 +0000 (00:16 +0100)
When running f3706b0b87, the following command gets sent to redis:
  "ZADD" "collectd/hostname/entropy/entropy" "1415487432.000000" "1415487432:932"
Meaning the value actually stored, and later returned by redis is:
"<timstamp>:<value>".

b7984797 accidentally dropped the comma separating the timestamp and the
value, which leads the plugin to store a somewhat confusing value in
redis:
  "ZADD" "collectd/hostname/entropy/entropy" "1415487432.000000" "1415487432932"

src/write_redis.c

index 58f2cae3e28dc02b3a9fad6bcbb80171230631a3..1d9ec27c0b60bf43900e9504e4271ce12a623a70 100644 (file)
@@ -84,7 +84,7 @@ static int wr_write (const data_set_t *ds, /* {{{ */
   }                                                                  \
 } while (0)
 
-  APPEND ("%lu", (unsigned long) vl->time);
+  APPEND ("%lu:", (unsigned long) vl->time);
   for (i = 0; i < ds->ds_num; i++)
   {
     if (ds->ds[i].type == DS_TYPE_COUNTER)