----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57451/#review172097 -----------------------------------------------------------
ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase.py Line 115 (original) <https://reviews.apache.org/r/57451/#comment245174> Just curious, did someone from the Hbase project confirm this change? - Alejandro Fernandez On April 14, 2017, 4:52 a.m., zhangxiaolu zhangxiaolu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57451/ > ----------------------------------------------------------- > > (Updated April 14, 2017, 4:52 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 > 20894ec > > > Diff: https://reviews.apache.org/r/57451/diff/6/ > > > 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 > >