jpountz commented on PR #12426: URL: https://github.com/apache/lucene/pull/12426#issuecomment-1629645539
I must say I'm surprised it's being considered for usage on Amazon Product Search: `IndexOrDocValuesQuery` should pick the best query, and this `VerifyingQuery` would run both queries, so it would make this query at least 2x slower, probably closer to 10x slower on average. In extreme cases, the doc values query will run a linear scan over the entire dataset, so this approach looks quite extreme. This kind of verification feels like a better fit for unit tests than for a subset of production? -- 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
