atris edited a comment on issue #1303: LUCENE-9114: Improve ValueSourceScorer's 
Default Cost Implementation
URL: https://github.com/apache/lucene-solr/pull/1303#issuecomment-593133186
 
 
   @dsmiley Thinking further, I see no obvious way of ValueSourceScorer being 
able to determine a reasonable cost without having inputs from FunctionValues, 
and currently, the sane way of getting a cost out of FunctionValues is through 
its TPI (unless I am missing something?).
   
   The best cost metrics will come when specific implementations (such as 
IntFieldSource) expose their cost by internally evaluating their complexity in 
a source specific manner instead of delegating to the default FV matchCost 
implementation.
   
   Maybe have FunctionValues expose an abstract cost() method, have all FV 
derivatives implement it and then simply let VSC's matchCost use that method?
   
   (and oh, I realised that this PR is definitely wrong, for some reason I 
missed that FV delegates to VSC using itself as the delegated FV value. Need to 
add Lucene tests for VSC...)

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to