redis plugin: Remove `operations_per_second` metric.
authorPavel Rochnyack <pavel2000@ngs.ru>
Sun, 1 Jul 2018 06:52:31 +0000 (13:52 +0700)
committerPavel Rochnyack <pavel2000@ngs.ru>
Sun, 1 Jul 2018 06:52:41 +0000 (13:52 +0700)
commit85fd55f0c419cfd23aef9a408e8a0167aca9e4d5
treec2bb3fb4e530eef348e62244cee14669b0426c18
parentb626c5956e7289cb3d7fceb230baba0182f36d68
redis plugin: Remove `operations_per_second` metric.

The `operations_per_second` metric removed as duplicate of other metric and as
it has wrong value also. Collectd should not report wrong values.

Metric `operations_per_second` is based on `instantaneous_ops_per_sec` field
from Redis `INFO` output. That field is calculated based on buffer of 16 values,
updated from `stat_numcommands` once per 100ms.
So, `instantaneous_ops_per_sec` value has average only for last 1.6 seconds.
While Collectd poll period is 10 seconds, statistics for 8.4 seconds is lost.

Herewith, Collectd already has correct metric `total_operations`, based on
`total_commands_processed` field taken from same `stat_numcommands` internally
in Redis. Value of this metric should be used instead of removed metric.
src/redis.c