dsmiley commented on PR #4811: URL: https://github.com/apache/solr/pull/4811#issuecomment-5510740809
Looking for a second opinion @gerlowskija . This PR removed some (non-deprecated) SolrJ classes/methods that implied the use of (deprecated) ContentStream. IMO those didn't get much usage but it's not nothing. If this doesn't come to 10.x, then this stays in 11. If it stays in 11, ironically it implies this PR could have been more aggressive -- actually move ContentStream. If it stays in 11, we could nonetheless have a follow-up PR to *add* the public API additions introduced here so that 10.1+ users can shift over to alternatives before 11. -- 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]
