zhang-arvin opened a new pull request, #12537:
URL: https://github.com/apache/gravitino/pull/12537
## What changes were proposed in this pull request?
Fix the empty classpath entry issue in `common.sh`. When
`GRAVITINO_CLASSPATH` is initially unset, the previous code
`GRAVITINO_CLASSPATH+=":${GRAVITINO_CONF_DIR}"` would produce a classpath
starting with a colon separator, resulting in an empty entry that Java
interprets as the current working directory.
This PR changes the classpath initialization to check if
`GRAVITINO_CLASSPATH` is already set before appending the config directory.
## Why are the changes needed?
Fixes #12444. An empty classpath entry allows classes from the working
directory to be loaded by the application classloader, which can shadow
dependencies from catalog classloaders.
## Does this PR introduce any user-facing change?
No.
--
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]
