----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44792/#review123458 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog222.java (line 90) <https://reviews.apache.org/r/44792/#comment185708> Minor. Most of these variables will not be required after this change. - Aravindan Vijayan On March 14, 2016, 5:25 p.m., Dmytro Sen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44792/ > ----------------------------------------------------------- > > (Updated March 14, 2016, 5:25 p.m.) > > > Review request for Ambari, Aravindan Vijayan and Sid Wagle. > > > Bugs: AMBARI-15402 > https://issues.apache.org/jira/browse/AMBARI-15402 > > > Repository: ambari > > > Description > ------- > > TTL should be set in seconds for all the tables and be greater than 0. > timeline.metrics.host.aggregator.daily.ttl was = 0.0 after upgrade. > > > Diffs > ----- > > > ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/PhoenixHBaseAccessor.java > 611cb92 > > ambari-metrics/ambari-metrics-timelineservice/src/test/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/ITPhoenixHBaseAccessor.java > 93ba770 > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog222.java > b3ae237 > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/configuration/ams-site.xml > fd1c2ac > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/stack_advisor.py > 6c62f0f > > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog222Test.java > 3712994 > ambari-server/src/test/python/stacks/2.2/common/test_stack_advisor.py > 8d8938f > > Diff: https://reviews.apache.org/r/44792/diff/ > > > Testing > ------- > > Unit tests passed > Tested end-to-end > > > Thanks, > > Dmytro Sen > >