benwtrent commented on PR #13407:
URL: https://github.com/apache/lucene/pull/13407#issuecomment-2148390886

   > For HNSW efSearch is a core parameters during search time. This is 
convenient for users to not have to have the logic to strip off top k values on 
their end.
   
   I understand, but this is trivial to do yourself by collecting and rewriting 
the query directly.
   
   > Why not Lucene have the support for something that is a core parameter, 
when its cheap enough to have it so users have the option to choose how they 
want to use it?
   
   While it is cheap, it is a further coupling of kNN to HNSW. Now the user 
interface itself when querying assumes "HNSW-esque" things. Instead of a simple 
"Give me k values" its now "give me k values with this particular HNSW 
parameter". 
   
   There has been talk of adding a "flat" codec to Lucene, that simply takes 
advantage of quantization and stores the vectors not in the HNSW graph. In that 
instance, what would `efSearch` mean?


-- 
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

Reply via email to