Github user sansanichfb commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1291#discussion_r141685224
--- Diff: pxf/pxf-service/src/main/resources/pxf-log4j.properties ---
@@ -23,7 +23,7 @@ log4j.rootLogger=INFO, ROLLINGFILE
log4j.appender.ROLLINGFILE=org.apache.log4j.RollingFileAppender
# use the same value directory as set for $PXF_LOGDIR
-log4j.appender.ROLLINGFILE.File=/var/log/pxf/pxf-service.log
+log4j.appender.ROLLINGFILE.File=$${env:$PXF_LOGDIR}/pxf-service.log
--- End diff --
According to docs, Log4J supports both environment and system vars,
$${env:USER} and ${sys:logPath} respectively.
---
