cpoerschke commented on PR #2524: URL: https://github.com/apache/solr/pull/2524#issuecomment-2197018871
> (no CHANGES.txt; just deprecating low level stuff & refactoring stuff that shouldn't have an effect) > > WDYT @cpoerschke Perhaps `QueryComponent` and `SolrIndexSearcher` might not be totally low level stuff here, i.e. folks with custom components (or just a copy/paste/adapt 'fork' of QueryComponent) might call the APIs being deprecated. Therefore it could be nice to "Other Changes" mention about the deprecation before it goes away in Solr 10 or so. The `@deprecated` annotation serves the same purpose but only slightly so i.e. they might only get noticed when users try to upgrade whereas a mention in the CHANGES.txt might trigger an earlier realisation of "hey, we need to do some prep work or keep this in mind when upgrading our custom query component in future" etc. -- 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
