slow-J opened a new pull request, #12893: URL: https://github.com/apache/lucene/pull/12893
Following up on @mikemccand's comment in previous CheckIndex PR:https://github.com/apache/lucene/pull/12876. > 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! Targetting Lucene 9.10. -- 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
