pradeepagrawal8184 commented on code in PR #965:
URL: https://github.com/apache/ranger/pull/965#discussion_r3289660108
##########
ranger-plugin-classloader/src/main/java/org/apache/ranger/plugin/classloader/RangerPluginClassLoader.java:
##########
@@ -40,6 +40,23 @@
public class RangerPluginClassLoader extends URLClassLoader {
Review Comment:
Agreed that changing RangerPluginClassLoader globally is heavy. For Solr 9,
Jetty already provides Jersey on the webapp classpath, so we can avoid bundling
Jersey/HK2 in ranger-solr-plugin-impl and rely on the existing
component-classloader delegation (revert the classloader change).
The classloader isolation was only needed because the plugin tarball still
contained Jersey 2.39.1 while ranger-plugins-common also pulled 2.47, and
delegation mixed classes across loaders. The fix is to exclude all
org.glassfish.jersey.* / org.glassfish.hk2.* from the Solr plugin assembly (not
ship duplicate jars), not to remove Jersey from Solr itself.
I’ll validate policy refresh (RangerAdminRESTClient) and Kerberos/docker
Solr startup after dropping packaged Jersey and reverting the classloader
change.
--
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]
