----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23854/#review48647 -----------------------------------------------------------
We probably should remove old definitions like metrics/dfs/FSNamesystem/CapacityUsedGB and metrics/dfs/FSNamesystem/CapacityTotalGB if we are replacing them with more precise metrics - Dmitro Lisnichenko On July 24, 2014, 2:19 p.m., Dmytro Sen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/23854/ > ----------------------------------------------------------- > > (Updated July 24, 2014, 2:19 p.m.) > > > Review request for Ambari, Alexandr Antonenko and Dmitro Lisnichenko. > > > Bugs: AMBARI-6578 > https://issues.apache.org/jira/browse/AMBARI-6578 > > > Repository: ambari > > > Description > ------- > > Switch Dashboard widget, HDFS > Services metrics chart, and other places in > UI where we were using the ***GB version of metrics (such as CapacityTotalGB > or CapacityUsedGB) to these instead, that are in bytes, providing more > precision: > CapacityTotal > CapacityUsed > CapacityRemaining > CapacityNonDFSUsed > Note: when making this switch, need to do the units conversion so values are > easy to read. > > > Diffs > ----- > > > ambari-server/src/main/resources/stacks/HDP/2.0.6.GlusterFS/services/HDFS/metrics.json > c52a4c0 > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/metrics.json > be49109 > > ambari-server/src/test/resources/stacks/HDP/2.0.5/services/HDFS/metrics.json > fe33f71 > > ambari-server/src/test/resources/stacks/HDP/2.0.7/services/HDFS/metrics.json > be49109 > > ambari-server/src/test/resources/stacks/HDP/2.0.8/services/HDFS/metrics.json > f33a0c0 > ambari-web/app/messages.js 537b5f5 > ambari-web/app/utils/ajax/ajax.js 976459a > ambari-web/app/views/main/service/info/metrics/hdfs/space_utilization.js > 6bf00f0 > > Diff: https://reviews.apache.org/r/23854/diff/ > > > Testing > ------- > > not applicable > > > Thanks, > > Dmytro Sen > >
