Github user sansanichfb commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1294#discussion_r142762535
--- 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 --
There is no guaranty that that group exists, so we don't have to set it
here.
---
