[GitHub] incubator-hawq issue #1301: Re-added pxf profile default to rpm and tar task...

2017-10-13 Thread shivzone
Github user shivzone commented on the issue: https://github.com/apache/incubator-hawq/pull/1301 After some investigation, the webapp does have pxf-profiles-default.xml file in pxf-service/webapps/pxf/WEB-INF/classes/ along with other configuration files. @lavjain is right, we dont'

[GitHub] incubator-hawq issue #1301: Re-added pxf profile default to rpm and tar task...

2017-10-13 Thread lavjain
Github user lavjain commented on the issue: https://github.com/apache/incubator-hawq/pull/1301 If pxf-private-default is not needed, then we should remove it. The less number of conf files we have, the better. ---

[GitHub] incubator-hawq issue #1301: Re-added pxf profile default to rpm and tar task...

2017-10-12 Thread shivzone
Github user shivzone commented on the issue: https://github.com/apache/incubator-hawq/pull/1301 @lavjain pxf-profiles.xml is better to be empty to prevent users from incorrectly updating the default entires in it. This is similar to the separation we have between