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

   ### What changes were proposed in this pull request?
   
   This PR updates `IsolatedClassLoader` to resolve simple class names from 
`java.lang`, such as `Override`, before falling back to the existing 
isolated/shared class loading logic.
   
   It also adds a unit test to verify that loading `Override` through 
`IsolatedClassLoader` resolves to `java.lang.Override`.
   
   ### Why are the changes needed?
   
   Some generated-code paths may ask the context classloader to resolve simple 
`java.lang` names instead of fully qualified names. For example, Janino/codegen 
usage can try to resolve `Override` directly.
   
   Without this fallback, Gravitino's isolated catalog classloader throws 
`ClassNotFoundException` for these simple names, which can break compatibility 
with libraries that perform runtime code generation.
   
   Fix: #12014
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   Added unit test coverage in `TestIsolatedClassLoader`.
   
   Also ran:
   
   ```bash
   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