adriancole edited a comment on pull request #5818:
URL: https://github.com/apache/incubator-pinot/pull/5818#issuecomment-670817753


   fyi I'm currently inlining a script as when you consider docker knows the 
only image it will run on, the script doesn't need to do work like detect OS, 
JVM etc.
   
   In this case, I'm ok closing this out if it isn't working.
   
   ```sh
   #!/bin/sh
   # For java 9 and later version, we need to explicitly set Pinot Plugins 
directory into classpath.
   CLASSPATH=$(find lib plugins -type f|xargs echo|tr ' ' ':')
   # TODO: link to pinot issue why we need internal access
   exec java \
     --add-opens java.base/jdk.internal.ref=ALL-UNNAMED \
     -classpath $CLASSPATH \
     $JAVA_OPTS \
     -Dlog4j.configurationFile=etc/log4j2.properties \
     -Dapp.name=pinot-admin \
     -Dapp.pid=$$ \
     -Dapp.repo=$PINOT_HOME/lib \
     -Dapp.home=$PINOT_HOME \
     -Dbasedir=$PINOT_HOME \
     org.apache.pinot.tools.admin.PinotAdministrator \
     "$@"
   ```


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to