----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42173/#review113968 -----------------------------------------------------------
ambari-web/test/controllers/wizard/step7_test.js (line 1172) <https://reviews.apache.org/r/42173/#comment174751> Please change this to the exact method you are testing, since you have refactored the method: removeHawqStandbyHostAddressConfig - Matt On Jan. 11, 2016, 5:08 p.m., bhuvnesh chaudhary wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42173/ > ----------------------------------------------------------- > > (Updated Jan. 11, 2016, 5:08 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Jaimin Jetly, jun aoki, > Jayush Luniya, Oleksandr Diachenko, Richard Zang, and Sumit Mohanty. > > > Bugs: AMBARI-14599 > https://issues.apache.org/jira/browse/AMBARI-14599 > > > Repository: ambari > > > Description > ------- > > Update hawq configs to remove hawq_standby_address_host on single node > clusters. > Currently, its available by default in templates, so is picked up by UI and > is pushed to the filesystem during installation. On single node cluster, this > property is not required to be displayed on UI and should not be pushed to > filesystem. > > > Diffs > ----- > > ambari-web/app/controllers/wizard/step7_controller.js 42b5b87 > ambari-web/test/controllers/wizard/step7_test.js a6a5fab > > Diff: https://reviews.apache.org/r/42173/diff/ > > > Testing > ------- > > yes. > 22965 tests complete (19 seconds) > 163 tests pending > > [INFO] BUILD SUCCESS > > > Thanks, > > bhuvnesh chaudhary > >
