gianm commented on code in PR #12570:
URL: https://github.com/apache/druid/pull/12570#discussion_r887314259


##########
examples/bin/run-druid:
##########
@@ -42,7 +42,7 @@ if [ -z "$JAVA_BIN" ]; then
   exit 1
 fi
 
-LOG_DIR="${DRUID_LOG_DIR:=${WHEREAMI}/log}"
+LOG_DIR="${DRUID_LOG_DIR:=${WHEREAMI}/../log}"

Review Comment:
   @FrankChen021 just wanted to double check with you to see if I am missing 
anything here. I think the `../` ends up doing more what you intended in 
https://github.com/apache/druid/pull/10147 anyway.
   
   (Since `$WHEREAMI` is the `bin` directory, `$WHEREAMI/..` is the Druid 
distribution directory.)



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to