benwtrent commented on PR #14085: URL: https://github.com/apache/lucene/pull/14085#issuecomment-2591446496
Thanks @benchaplin Those constants and numbers are focused on expanding and contracting the graph search as we hit various NSW with more or fewer matching docs. One dictates how many extra matching candidates we take over the current NSW connections. (e.g. collecting 40 valid candidates instead of 32). The other is controlling how far into the graph we will search non- matching candidates. At some point, you just gotta cut your losses. I tried to relate this to how restricted the filter is respective to the NSW connections. The numbers and their calculations are definitely not written in stone, so if you find something that works better let me know. Definitely a POC and poorly documented right now. Another good idea is to expand our entry points to layer zero on very restricted filters (e.g. when the filter limit is less than the number of docs on layer 1 or something). -- 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