----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46310/#review129405 -----------------------------------------------------------
ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_interactive.py (line 85) <https://reviews.apache.org/r/46310/#comment192830> I'm actually going to change this since we need to populate 2 dirs, hive conf and the conf.server subdir. ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/params_linux.py (line 497) <https://reviews.apache.org/r/46310/#comment192831> Ideally, we should be using /etc/tez_hive2/$version/0 or /usr/hdp/current/****/conf to avoid any confusion of symlinks. - Alejandro Fernandez On April 18, 2016, 2:11 a.m., Sumit Mohanty wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46310/ > ----------------------------------------------------------- > > (Updated April 18, 2016, 2:11 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Jaimin Jetly, and Swapan > Shridhar. > > > Bugs: AMBARI-15937 > https://issues.apache.org/jira/browse/AMBARI-15937 > > > Repository: ambari > > > Description > ------- > > Along with the changes, there are some clean-up and addition of basic unit > tests. Some more comments in the diff. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive.py > 7d913e5 > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_interactive.py > 9d05d37 > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/params_linux.py > 03fd40c > ambari-server/src/test/python/stacks/2.5/HIVE/test_hive_server_int.py > PRE-CREATION > ambari-server/src/test/python/stacks/2.5/configs/hsi_default.json > PRE-CREATION > > Diff: https://reviews.apache.org/r/46310/diff/ > > > Testing > ------- > > Ran unit tests manually, and tested by applying patch. > > > Thanks, > > Sumit Mohanty > >
