jpountz commented on PR #14333: URL: https://github.com/apache/lucene/pull/14333#issuecomment-2776835446
> I thought i only need to fork a Lucene103BlockTreeTerms to intersect with Lucene101Postings. But it seems challenging based on current API design. I have to fork the new Lucene103Postings as well. Maybe this is a point can be improved? Right, this is because a Lucene `PostingsFormat` encapsulates both the format of the terms dictionary and of postings lists. I guess that we could make terms dictionaries and postings formats pluggable, but that could make it harder to do interesting things such as inlining data about postings into the terms dictionary. > 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. -- 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
