----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22637/#review45784 -----------------------------------------------------------
Ship it! Ship It! - Dmitro Lisnichenko On June 16, 2014, 4:59 p.m., Andrew Onischuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/22637/ > ----------------------------------------------------------- > > (Updated June 16, 2014, 4:59 p.m.) > > > Review request for Ambari and Dmitro Lisnichenko. > > > Bugs: AMBARI-6142 > https://issues.apache.org/jira/browse/AMBARI-6142 > > > Repository: ambari > > > Description > ------- > > I changed Hive configs on a single node cluster. 4 components were correctly > shown as stale - needing restart. I went to the host page and I see that all > hive components (including hive-client) need restart. I restarted only the > _Hive Metastore_ component which incorrectly removed _Hive Client_ stale > indicator. > > This is problematic because _Hive Client_ and _Hive Metastore_ use two > different config folders (/etc/hive/conf and /etc/hive/conf.server > respectively), and _Hive Client_ never updates its configs. > > > Diffs > ----- > > > ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HIVE/package/scripts/hive.py > 2efb5e2 > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HIVE/package/scripts/hive.py > 1c65439 > ambari-server/src/test/python/stacks/1.3.2/HIVE/test_hive_client.py b0d7605 > ambari-server/src/test/python/stacks/1.3.2/HIVE/test_hive_metastore.py > 31952ae > ambari-server/src/test/python/stacks/1.3.2/HIVE/test_hive_server.py 6cd9f52 > ambari-server/src/test/python/stacks/2.0.6/HIVE/test_hive_client.py 15b692a > ambari-server/src/test/python/stacks/2.0.6/HIVE/test_hive_metastore.py > d9bbb09 > ambari-server/src/test/python/stacks/2.0.6/HIVE/test_hive_server.py cec393a > ambari-server/src/test/python/stacks/2.1/HIVE/test_hive_metastore.py > 5ab5744 > > Diff: https://reviews.apache.org/r/22637/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Andrew Onischuk > >
