diqiu50 commented on PR #11846: URL: https://github.com/apache/gravitino/pull/11846#issuecomment-4989046503
Although this change can solve the issue, it requires switching the class loader multiple times for every operation request, which is not a good design. I see two possible approaches: Set the class loader at a higher level in NormalizeDispatcher.java and wrap the entire operation with doWith, so the lower-level code does not need to handle class-loader switching. Let Capability hold the class loader and manage it with reference counting. This would prevent the class loader from being closed while it is still in use and would also solve the issue. However, this approach requires a larger change, so I would prefer not to use it for now. However, both approaches would require relatively large changes. -- 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]
