msokolov commented on PR #16489: URL: https://github.com/apache/lucene/pull/16489#issuecomment-5373632509
I'm not sure how I missed the factory change, sorry! As for the old constructor, this is a public method in a public class, and it's not marked @lucene.experimental, so we cannot remove it without a major release. Having said that, we could decide to make that change on main (targeting v. 11), and only add back the constructor if/when we backport the change to the 10.x branch from which we would cut any minor version release (10.6, etc). I don't have a strong opinion about that; it just seems easier to keep the old constructor. -- 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]
