Github user shivzone commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1294#discussion_r142801670
--- 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 --
700 is way too restrictive for log directory. This would mean only gpadmin
or user 'pxf' will have access to reading the logs. Lets jsut use the default
permission level instead of expclitly setting it.
---
