slow-J commented on PR #12876: URL: https://github.com/apache/lucene/pull/12876#issuecomment-1842557831
> I'm not sure about the removal of these two functions, they're quite nice, only taking a CodecReader and an InfoStream. Removing them is forcing users to call the more complex functions and understand what a level is, or what the boolean means. Other changes look good to me. Thanks @jpountz, will add these functions back in then. I was wondering about the use of `@lucene.experimental` throughout this class. It has been experimental for > 10 years. Will the entire class or some parts, ever be classified as non-experimental? -- 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...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
