----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42536/#review115698 -----------------------------------------------------------
ambari-server/src/main/resources/common-services/HAWQ/2.0.0/package/scripts/hawq_constants.py (line 63) <https://reviews.apache.org/r/42536/#comment176715> Why gpadmin user is hardcoded? Shouldn't we use hawq_constants.hawq_user variable instead? - Oleksandr Diachenko On Jan. 20, 2016, 6:34 p.m., Matt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42536/ > ----------------------------------------------------------- > > (Updated Jan. 20, 2016, 6:34 p.m.) > > > Review request for Ambari, Alexander Denissov, bhuvnesh chaudhary, jun aoki, > Lav Jain, and Newton Alex. > > > Bugs: AMBARI-14732 > https://issues.apache.org/jira/browse/AMBARI-14732 > > > Repository: ambari > > > Description > ------- > > Current HAWQ service check implementation executes tests with HAWQ internal > tables. Extend the service check to include tests with external tables on > HDFS, which can be queried through PXF > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/HAWQ/2.0.0/package/scripts/hawq_constants.py > fe3743b > > ambari-server/src/main/resources/common-services/HAWQ/2.0.0/package/scripts/params.py > aa3527c > > ambari-server/src/main/resources/common-services/HAWQ/2.0.0/package/scripts/service_check.py > c35fdce > > ambari-server/src/main/resources/common-services/HAWQ/2.0.0/package/scripts/utils.py > c1a38bb > > Diff: https://reviews.apache.org/r/42536/diff/ > > > Testing > ------- > > Manually Tested > > > Thanks, > > Matt > >
