msokolov commented on PR #14178: URL: https://github.com/apache/lucene/pull/14178#issuecomment-2807590241
I don't think we should block on the luceneutil improvements. Do we have consensus that this should become the default as implemented by AbstractKnnVectorQuery? We've done some testing here, but this really hasn't had a chance to bake in anybody's production environment. I'll propose that we commit this more or less as is, as a new Query type. We can give people a chance to run with it, use in their query factories or whatever, perhaps making it the default in some environments, and once we're happy with its battle-testedness we can fold it in as the default, deprecating the old way with cross-thread communication. Then we can backport to 10.x -- 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
