yuqi1129 opened a new pull request, #11222:
URL: https://github.com/apache/gravitino/pull/11222

   ### What changes were proposed in this pull request?
   
   This PR scopes launcher PID detection to the current `GRAVITINO_HOME` in:
   
   - `bin/gravitino.sh.template`
   - `bin/gravitino-iceberg-rest-server.sh.template`
   - `bin/gravitino-lance-rest-server.sh.template`
   
   The PID lookup now matches both the server process name and the current 
installation path, and uses `ps xww` to avoid truncated command lines.
   
   ### Why are the changes needed?
   
   The old lookup only matched the server class name, so sibling Gravitino 
installations on the same host could be mistaken for the current installation. 
This could make `start`, `stop`, `restart`, and `status` operate on the wrong 
JVM.
   
   Fix: #11221
   
   ### Does this PR introduce _any_ user-facing change?
   
   Yes. Launcher scripts now detect only JVMs started from the same Gravitino 
installation directory.
   
   ### How was this patch tested?
   
   - `bash -n bin/gravitino.sh.template 
bin/gravitino-iceberg-rest-server.sh.template 
bin/gravitino-lance-rest-server.sh.template`
   - `git diff --check`


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

Reply via email to