adriancole commented 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
   # TODO: link to pinot issue why we need internal access
   exec java \
     --add-opens java.base/jdk.internal.ref=ALL-UNNAMED \
     -classpath 'lib/*' \
     $JAVA_OPTS \
     -Dapp.name=pinot-admin \
     -Dapp.pid=$$ \
     -Dapp.repo=$PINOT_HOME/lib \
     -Dapp.home=$PINOT_HOME \
     -Dbasedir=$PINOT_HOME \
     -Dplugins.dir=$PINOT_HOME/plugins \
     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