> On Dec. 1, 2015, 9:48 p.m., jun aoki wrote: > >
More information regarding this change: - Location of initdb has been moved to logs directory as a result of https://issues.apache.org/jira/browse/HAWQ-33 - Please refer https://github.com/apache/ambari/blob/trunk/ambari-server/src/main/resources/common-services/HAWQ/2.0.0/package/scripts/utils.py#L34-#L38 which creates the hawq_segment_dir as hawq_user. The call to this function is made here https://github.com/Pivotal-Hadoop/ambari/blob/trunk/ambari-server/src/main/resources/common-services/HAWQ/2.0.0/package/scripts/hawqsegment.py#L84 - Matt ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40686/#review108608 ----------------------------------------------------------- On Nov. 24, 2015, 5:08 p.m., Matt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40686/ > ----------------------------------------------------------- > > (Updated Nov. 24, 2015, 5:08 p.m.) > > > Review request for Ambari. > > > Bugs: AMBARI-14054 > https://issues.apache.org/jira/browse/AMBARI-14054 > > > Repository: ambari > > > Description > ------- > > HAWQ service gives the ownership of parent directories of hawq_master_dir and > hawq_segment_dir to hawq_user. > > This is not ideal because the parent directory might be / (root directory), > and root directory ownership should not be given to hawq_user > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/HAWQ/2.0.0/package/scripts/hawqsegment.py > 0e349d2 > > ambari-server/src/main/resources/common-services/HAWQ/2.0.0/package/scripts/master_helper.py > fdfd7c9 > > Diff: https://reviews.apache.org/r/40686/diff/ > > > Testing > ------- > > Manually tested > > After changes were made, HAWQ was deployed on a cluster, and the directory > ownership and permissions were verified for hawq_master_dir and > hawq_segment_dir > > > Thanks, > > Matt > >