----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38156/#review97925 -----------------------------------------------------------
Only comment: What about tolerance for old data that was value * 100? Is there any way to checkpoint? - Sid Wagle On Sept. 7, 2015, 2:18 p.m., Dmytro Sen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38156/ > ----------------------------------------------------------- > > (Updated Sept. 7, 2015, 2:18 p.m.) > > > Review request for Ambari, Myroslav Papirkovskyy, Sid Wagle, and Vitalyi > Brodetskyi. > > > Bugs: AMBARI-13024 > https://issues.apache.org/jira/browse/AMBARI-13024 > > > Repository: ambari > > > Description > ------- > > Some metrics are double-edited(on MetricsDataTransferMethodFactory and > sink/monitor side) > For instance, cpu-metrics are multiplied by 100 on monitor side, but divided > by MetricsDataTransferMethodFactory > > > Diffs > ----- > > > ambari-metrics/ambari-metrics-host-monitoring/src/main/python/core/host_info.py > e0ed8e7 > > ambari-metrics/ambari-metrics-host-monitoring/src/test/python/core/TestHostInfo.py > bdedc2e > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractPropertyProvider.java > f265c4f > > ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/MetricReportingAdapter.java > ad711f1 > > ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/MetricsDataTransferMethod.java > 004742e > > ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/MetricsDataTransferMethodFactory.java > 9d70158 > > ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/MetricsDownsamplingMethod.java > 8589a1b > > ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/MetricsDownsamplingMethodFactory.java > 19df14c > > Diff: https://reviews.apache.org/r/38156/diff/ > > > Testing > ------- > > In progress > > > Thanks, > > Dmytro Sen > >
