mikemccand commented on PR #12876: URL: https://github.com/apache/lucene/pull/12876#issuecomment-1845176680
> 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? LOL 10 years of experimental API! I bet Lucene has even older experimental APIs :) Too bad we don't have an "experimental API age reporting tool"... Yeah this is a great question, when to remove the experimental API tag. We have no well defined process -- it's just devs (like you, thanks!) noticing and raising the question on a case by case basis. I do think some of these tags in `CheckIndex.java` could be removed, e.g. on each of the `testXXX` methods? But likely we should leave it on each of the `XXXStatus` returned classes, so we are free to break/change what metadata these classes return? And let's do it separately -- this PR looks like a great cleanup already. Progress not perfection! -- 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]
