Github user sansanichfb commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1294#discussion_r143025470
--- Diff: pxf/pxf-service/src/scripts/pxf-service ---
@@ -267,15 +280,19 @@ function commandWebapp()
function createLogsDir()
{
mkdir -p $PXF_LOGDIR
- chown -R $instance_owner $PXF_LOGDIR
+ if $allow_custom_pxf_user; then
+ chown -R $instance_owner $PXF_LOGDIR
+ fi
chmod 700 $PXF_LOGDIR
--- End diff --
I would prefer to keep it restrictive and change to less restrictive if
needed later.
---
