mikemccand commented on PR #15616: URL: https://github.com/apache/lucene/pull/15616#issuecomment-3820129008
> @mikemccand and @uschindler Thanks for the background information. I agree that these classes eventually need to removed, but this would be a bigger effort (there are many tests and other classes that depend on them). > > For the time being, to make nightly tests pass, are we ok to go with the approach in this PR? Sorry for slow response -- yeah +1 to merge this approach, and defer the "garbage collection" to remove tests that cover only way-old formats that we no longer even support reading. Maybe open spinoff for this? Might be a fun thing to try with Claude Code or Gemini CLI :) -- 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]
