Github user lavjain commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1294#discussion_r142551336
--- Diff: pxf/pxf-service/src/scripts/pxf-service ---
@@ -72,9 +77,11 @@ function createInstance()
return 1
fi
- chown -R $instance_owner $instance_root
- chmod 700 $instance_root/$instance_name
- return 0
+ if $allow_custom_pxf_user; then
+ chown -R $instance_owner $instance_root
--- End diff --
Should we use chown -R $instance_owner:$instance_owner instead (to include
group)?
---
