From: Sebastian Harl <sh@tokkee.org>
Date: Sun, 23 Nov 2014 15:56:08 +0000 (+0100)
Subject: timeseries::rrdtool: Limit to 1000 data-points for now.
X-Git-Tag: sysdb-0.7.0~125
X-Git-Url: https://git.tokkee.org/?p=sysdb.git;a=commitdiff_plain;h=6eee9bc367946ada5b0596c6029a096ea5332a9f

timeseries::rrdtool: Limit to 1000 data-points for now.

That's good enough for most use cases (ought to be enough for anybody!) and
avoids excessive memory use when requesting data.
---

diff --git a/src/plugins/timeseries/rrdtool.c b/src/plugins/timeseries/rrdtool.c
index db24903..8ea734f 100644
--- a/src/plugins/timeseries/rrdtool.c
+++ b/src/plugins/timeseries/rrdtool.c
@@ -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];