----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39491/#review103348 -----------------------------------------------------------
ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py (line 41) <https://reviews.apache.org/r/39491/#comment161363> Add check for HAWQ service is installed and short-circuit return if not installed. - Jayush Luniya On Oct. 20, 2015, 7:18 p.m., Alexander Denissov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39491/ > ----------------------------------------------------------- > > (Updated Oct. 20, 2015, 7:18 p.m.) > > > Review request for Ambari, Alejandro Fernandez, jun aoki, Jayush Luniya, > Mahadev Konar, and Oleksandr Diachenko. > > > Bugs: AMBARI-13442 > https://issues.apache.org/jira/browse/AMBARI-13442 > > > Repository: ambari > > > Description > ------- > > Introduced layout recommendations and validations for Apache HAWQ service > master components. > > Test cases use new test json files for input rather than hardcoding data in > the test case directly. > > > Diffs > ----- > > ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py > a2bae0f > ambari-server/src/test/python/stacks/2.3/common/hosts-1-host.json > PRE-CREATION > ambari-server/src/test/python/stacks/2.3/common/hosts-3-hosts.json > PRE-CREATION > ambari-server/src/test/python/stacks/2.3/common/services-hawq-1-host.json > PRE-CREATION > ambari-server/src/test/python/stacks/2.3/common/services-hawq-3-hosts.json > PRE-CREATION > > ambari-server/src/test/python/stacks/2.3/common/services-master_ambari_colo-3-hosts.json > PRE-CREATION > > ambari-server/src/test/python/stacks/2.3/common/services-master_standby_colo-3-hosts.json > PRE-CREATION > > ambari-server/src/test/python/stacks/2.3/common/services-nohawq-3-hosts.json > PRE-CREATION > > ambari-server/src/test/python/stacks/2.3/common/services-normal-hawq-3-hosts.json > PRE-CREATION > > ambari-server/src/test/python/stacks/2.3/common/services-normal-nohawq-3-hosts.json > PRE-CREATION > > ambari-server/src/test/python/stacks/2.3/common/services-standby_ambari_colo-3-hosts.json > PRE-CREATION > ambari-server/src/test/python/stacks/2.3/common/test_stack_advisor.py > e0c6d28 > > Diff: https://reviews.apache.org/r/39491/diff/ > > > Testing > ------- > > Added new and ran python unit tests: > ---------------------------------------------------------------------- > Total run:775 > Total errors:0 > Total failures:0 > OK > > > Thanks, > > Alexander Denissov > >
