Code

timeseries::rrdtool: Limit to 1000 data-points for now.
authorSebastian Harl <sh@tokkee.org>
Sun, 23 Nov 2014 15:56:08 +0000 (16:56 +0100)
committerSebastian Harl <sh@tokkee.org>
Sun, 23 Nov 2014 15:56:08 +0000 (16:56 +0100)
That's good enough for most use cases (ought to be enough for anybody!) and
avoids excessive memory use when requesting data.

src/plugins/timeseries/rrdtool.c

index db24903f5513785852a22748f05de88b1a4b0f91..8ea734fb643c42e8c518c7dbd51739635d5930c5 100644 (file)
@@ -106,6 +106,10 @@ sdb_rrd_fetch(const char *id, sdb_timeseries_opts_t *opts,
                rrd_freemem(data); \
        } while (0)
 
+       /* limit to about 1000 data-points for now
+        * TODO: make this configurable */
+       step = (end - start) / 1000;
+
        if (rrd_fetch_r(id, "AVERAGE", &start, &end, &step,
                                &ds_cnt, &ds_namv, &data)) {
                char errbuf[1024];