New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SOLR-16240: Fix kerberosPlugin module classloading #1546
SOLR-16240: Fix kerberosPlugin module classloading #1546
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Surprisingly simple workaround. Need to add exception to ForbiddenAPIs, and perhaps a code comment with JIRA link as a reminder to remove the workaround some day.
Yeah will get that stuff cleaned up. Not sure we will want to remove this workaround, it does let us use non-request centric plugins without worrying if those plugins use the thread classloader or not. (This is similar workaround we do for the sql module, which has libraries that will never accept a cloassloader probably...) I don't want to move merge this without an integration test, but I'm not sure how to get that working for kerberos... Might play around with that later if no one else has ideas. |
@janhoy added an integration test, which seems to work as expected, and fails when I remove the line that fixes this PR. Let me know if it looks good to you. |
LGTM |
Also added an integration test for the hadoop-auth module. (cherry picked from commit d9eba43)
Also added an integration test for the hadoop-auth module. (cherry picked from commit d9eba43)
https://issues.apache.org/jira/browse/SOLR-16240
The easy way out vs waiting on a hadoop fix and a hadoop release.