----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29788/#review67724 -----------------------------------------------------------
Ship it! Ship It! - Nate Cole On Jan. 12, 2015, 5:11 p.m., Alejandro Fernandez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29788/ > ----------------------------------------------------------- > > (Updated Jan. 12, 2015, 5:11 p.m.) > > > Review request for Ambari, Dmitro Lisnichenko, Jonathan Hurley, and Nate Cole. > > > Bugs: AMBARI-8917 > https://issues.apache.org/jira/browse/AMBARI-8917 > > > Repository: ambari > > > Description > ------- > > The prepare_rolling_restart() functions call copy_tarballs_to_hdfs(), which > gets the HDP version from the first component that matches the regex in the > output of hdp-select, which is incorrect. > Instead of getting the first component, it should query a specific component, > e.g., "hdp-select status hiveserver2" > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/functions/dynamic_variable_interpretation.py > 4f0dad3b308d325ec358a63c895e4708a6403943 > ambari-server/src/main/python/ambari-server.py > af27f5f3be7bf15597fb4545fcd7e393096ca4a7 > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/namenode_upgrade.py > d80d37755bece5e172e75535f3c9a5d3a1fa9b6c > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_server.py > e40f8c0ac4687052510e16639f683d0f19d315ac > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/webhcat.py > 9d53ea90b9e7e732a0ff0b8728cb12e898d3a4ac > > ambari-server/src/main/resources/common-services/PIG/0.12.0.2.0/package/scripts/service_check.py > a0e04abbe6c2eaede2ed8efbc79026bbc9d97135 > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/historyserver.py > 57bd37963047a7456d57a87f2032cd00a5cf8f9b > ambari-server/src/test/python/TestAmbariServer.py > 5f1fa388ff551222a534f4af1c00b34c070ec5fc > ambari-server/src/test/python/stacks/2.0.6/HIVE/test_hive_server.py > 5dcc016b2fffd8af9334ecba7e342108ae0a8116 > > Diff: https://reviews.apache.org/r/29788/diff/ > > > Testing > ------- > > Copied changed scripts to a new cluster, and restarted HiveServer2 and > HistoryServer to make sure that the logs showed hdp-select using that > component. > mvn clean test > > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 31:54.077s > [INFO] Finished at: Mon Jan 12 14:03:55 PST 2015 > [INFO] Final Memory: 67M/830M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Alejandro Fernandez > >
