senthh commented on PR #1669: URL: https://github.com/apache/solr/pull/1669#issuecomment-1568634452
> I'm not a fan of this solution. Yet a place of strings to maintain. > > What is wrong with the ClassNotFound exception you (probably) already get? Does it not tell you clearly the problem? Then you fix your class name and it works. If we should start adding such lists to all kind of plugins it would be a maintenance nightmare. I accept "ClassNotFound exception" is sufficient to find the issue. But problem arises when we upgrade from Solr 8(or lower) versions to Solr 9. In Solr 8 we have "org.apache.solr.security.*Plugin" as per doc, but in Solr 9 it got modified to use 'solr.*Plugin'. So we assume that what we configured is correct ,as per old document, if we miss this change, and continue with "org.apache.solr.security.*Plugin" security.json file. But I accept one has to go thro' the new documentation before upgradation. -- 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: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
