> On July 10, 2017, 5:08 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase.py > > Lines 197 (patched) > > <https://reviews.apache.org/r/60738/diff/1/?file=1772249#file1772249line197> > > > > This is already needed in HBase for 3.0 > > Please add a comment as well explaining why this was needed.
Typo, "This is also needed in HBase for 3.0" - Alejandro ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60738/#review180055 ----------------------------------------------------------- On July 10, 2017, 9:47 a.m., Andrew Onischuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60738/ > ----------------------------------------------------------- > > (Updated July 10, 2017, 9:47 a.m.) > > > Review request for Ambari, Dmitro Lisnichenko, Dmytro Sen, and Vitalyi > Brodetskyi. > > > Bugs: AMBARI-20950 > https://issues.apache.org/jira/browse/AMBARI-20950 > > > Repository: ambari > > > Description > ------- > > HBASE-17437 enables HBASE to put its rootdir on S3. Ambari calls HdfsResource > to create the dir for HBase, but it can not handle S3. > Fix should be similar to AMBARI-15157 for Hive. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase.py > 68856ce > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/params_linux.py > 43c6036 > > > Diff: https://reviews.apache.org/r/60738/diff/1/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Andrew Onischuk > >
