> On 三月 20, 2017, 5:31 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase.py > > Lines 59 (patched) > > <https://reviews.apache.org/r/57451/diff/1/?file=1659747#file1659747line59> > > > > Python unit tests are failing. > > Please see comment in Jira
I added some test and all test passed. - zhangxiaolu ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57451/#review169470 ----------------------------------------------------------- On 四月 10, 2017, 7:37 a.m., zhangxiaolu zhangxiaolu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57451/ > ----------------------------------------------------------- > > (Updated 四月 10, 2017, 7:37 a.m.) > > > Review request for Ambari, Dmytro Sen, Jaimin Jetly, Srimanth Gunturi, Sid > Wagle, and Yusaku Sako. > > > Bugs: AMBARI-20368 > https://issues.apache.org/jira/browse/AMBARI-20368 > > > Repository: ambari > > > Description > ------- > > The problem is caused by the wrong installation order, which means > hbase-client will be installed before the installation of hdfs-client. Under > such circumstance, installation of hdfs-client will fail because the > hadoop-conf-dir does not exist. > Two solutions can be provided to solve this problem: > 1. Creating the hadoop-conf-dir in hbase.py; > 2. Setting correct installation order in role_command_order.json; > Considering that hdfs-client may not beinstalled on the node especially when > adding new host, therefore, in my opinion, the second choice would be more > appropriate. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase.py > 8ad802e > ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_client.py > cffec06 > ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_master.py > 42bc989 > ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_regionserver.py > 9bb0dd7 > > ambari-server/src/test/python/stacks/2.0.6/HBASE/test_phoenix_queryserver.py > 60022e1 > > > Diff: https://reviews.apache.org/r/57451/diff/5/ > > > Testing > ------- > > done it > > > File Attachments > ---------------- > > AMBARI-20368.patch > > https://reviews.apache.org/media/uploaded/files/2017/04/07/e542fd73-b2b4-4542-94d2-151bbadd8b48__AMBARI-20368.patch > > > Thanks, > > zhangxiaolu zhangxiaolu > >