----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41151/#review110606 -----------------------------------------------------------
ambari-web/app/controllers/main/service/add_controller.js (line 414) <https://reviews.apache.org/r/41151/#comment170555> Please add Jaimin Jetly and Richard Zang to the code review. - Alejandro Fernandez On Dec. 10, 2015, 4:21 a.m., Lav Jain wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41151/ > ----------------------------------------------------------- > > (Updated Dec. 10, 2015, 4:21 a.m.) > > > Review request for Ambari, Alexander Denissov, Alejandro Fernandez, bhuvnesh > chaudhary, jun aoki, Matt, Newton Alex, and Oleksandr Diachenko. > > > Bugs: AMBARI-14301 > https://issues.apache.org/jira/browse/AMBARI-14301 > > > Repository: ambari > > > Description > ------- > > Looks like stackadvisor is used only upon a fresh install if no changes are > made to the master components. In that case, the client/slaves are installed > on all the hosts, so HAWQSEGMENT is co-located with DATANODE by default. > If HDFS is already installed and HAWQ is enabled using add services, then > Ambari chooses all hosts that do not have a MASTER component installed. This > determination is being done in the javascript code. > This patch is changing the hostlist for HAWQSEGMENT to be the same as > DATANODE in the latter case. > > > Diffs > ----- > > ambari-web/app/controllers/main/service/add_controller.js 1f8133d > > Diff: https://reviews.apache.org/r/41151/diff/ > > > Testing > ------- > > Manually tested. > > > Thanks, > > Lav Jain > >
