> On Sept. 7, 2015, 4:18 p.m., Sid Wagle wrote: > > Only comment: What about tolerance for old data that was value * 100? Is > > there any way to checkpoint? > > Dmytro Sen wrote: > In fact, no. > Only check every metric value if value > 100, which is very recource > consuming. > Moreover, even if value < 100, let say it's 0.7, there is no garantee > that it's an old value=70% it can be a new value=0.7%
What I meant was a checkpoint like version check to use the downsample or not. We cannot rely on the value alone that makes sense. - Sid ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38156/#review97925 ----------------------------------------------------------- On Sept. 7, 2015, 4:34 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, 4:34 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 > ambari-server/src/test/resources/ams/multiple_host_metrics.json 031e777 > ambari-server/src/test/resources/ams/single_host_metric.json 59ec685 > > Diff: https://reviews.apache.org/r/38156/diff/ > > > Testing > ------- > > In progress > > > Thanks, > > Dmytro Sen > >
