mccullocht commented on issue #15064: URL: https://github.com/apache/lucene/issues/15064#issuecomment-3309079250
I merged the change to add OSQ based codecs and I'm working on backwards codec changes but I've run into a bit of a dependency cycle: I can't move `Lucene99ScalarQuantizedVectorScorer` to `backwards_codecs` because it's linked directly by `DefaultVectorizationProvider`, but I can no longer test `Lucene99ScalarQuantizedVectorScorer` if I move all the remaining codec files to `backward_codecs` because they directly use the format (and presumably the writer). I'm not sure if this can be fixed without removing the off-heap scalar quantized scoring code which was just merged :(. I think this layering issue would likely repeat if we decided to use something besides OSQ or make really significant format changes there. cc @benwtrent @kaivalnp -- 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
