dsmiley commented on issue #1303: LUCENE-9114: Improve ValueSourceScorer's
Default Cost Implementation
URL: https://github.com/apache/lucene-solr/pull/1303#issuecomment-594728584
Okay; add to "Improvements" section.
This is
dsmiley commented on issue #1303: LUCENE-9114: Improve ValueSourceScorer's
Default Cost Implementation
URL: https://github.com/apache/lucene-solr/pull/1303#issuecomment-593675019
Ehh; nevermind my ill-thought-out idea of a cost on the Map context. There
are many ValueSource.getValues
dsmiley commented on issue #1303: LUCENE-9114: Improve ValueSourceScorer's
Default Cost Implementation
URL: https://github.com/apache/lucene-solr/pull/1303#issuecomment-593657932
Why a separate PR for my proposed test?
Your proposal is better than the status quo but I think is
dsmiley commented on issue #1303: LUCENE-9114: Improve ValueSourceScorer's
Default Cost Implementation
URL: https://github.com/apache/lucene-solr/pull/1303#issuecomment-593151099
Lets add the following test to TestFunctionRangeQuery:
```
@Test
public void testTwoRangeQueries()
dsmiley commented on issue #1303: LUCENE-9114: Improve ValueSourceScorer's
Default Cost Implementation
URL: https://github.com/apache/lucene-solr/pull/1303#issuecomment-593116806
Wouldn't this result in an infinite loop?