stillalex commented on PR #1892: URL: https://github.com/apache/solr/pull/1892#issuecomment-1712688382
I think is it not completely correct to say `Because this map is static`, because it was static before the change. I think the culprit is the extra verification that throws exception which can clash for tests running multiple instances in the same jvm (what was happening before, a silent override?). @cpoerschke this PR looks good to me. you were the one that merged SOLR-16282 so could you also take a look? what I think is still missing is a test showing the exception (and fix). -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
