mikemccand commented on PR #14333: URL: https://github.com/apache/lucene/pull/14333#issuecomment-2789601238
> > f we stick not to make it default, maybe this codec should be moved into test / sandbox / codec module? > > I don't have a strong opinion about making it the default right now in main or not, but @mikemccand seemed to care about not making it the default yet. In this case, I would suggest having it in lucene/core, hopefully in one or two weeks it becomes part of the default codec. +1 -- let's leave it in core. We should plan to make it the default in a couple weeks ... I'd just like to bake it a bit on `main` and see if Jenkins uncovers any fun surprises? With the gains it offers we should absolutely plan to use this in the default codec! -- 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
