[GitHub] [lucene-solr] mikemccand commented on a change in pull request #854: Shared PQ Based Early Termination for Concurrent Search

2019-09-11 Thread GitBox
mikemccand commented on a change in pull request #854: Shared PQ Based Early Termination for Concurrent Search URL: https://github.com/apache/lucene-solr/pull/854#discussion_r323174839 ## File path: lucene/core/src/java/org/apache/lucene/search/Collector.java ## @@ -77,4 +

[GitHub] [lucene-solr] iverase commented on a change in pull request #874: LUCENE-8976: Use exact distance between point and bounding rectangle in FloatPointNearestNeighbor

2019-09-11 Thread GitBox
iverase commented on a change in pull request #874: LUCENE-8976: Use exact distance between point and bounding rectangle in FloatPointNearestNeighbor URL: https://github.com/apache/lucene-solr/pull/874#discussion_r323177602 ## File path: lucene/sandbox/src/java/org/apache/lucene/do

[GitHub] [lucene-solr] iverase commented on a change in pull request #874: LUCENE-8976: Use exact distance between point and bounding rectangle in FloatPointNearestNeighbor

2019-09-11 Thread GitBox
iverase commented on a change in pull request #874: LUCENE-8976: Use exact distance between point and bounding rectangle in FloatPointNearestNeighbor URL: https://github.com/apache/lucene-solr/pull/874#discussion_r323177766 ## File path: lucene/sandbox/src/java/org/apache/lucene/do

[GitHub] [lucene-solr] iverase commented on issue #874: LUCENE-8976: Use exact distance between point and bounding rectangle in FloatPointNearestNeighbor

2019-09-11 Thread GitBox
iverase commented on issue #874: LUCENE-8976: Use exact distance between point and bounding rectangle in FloatPointNearestNeighbor URL: https://github.com/apache/lucene-solr/pull/874#issuecomment-530329820 In my test this approach looks faster than the previous approach, in particular for

[GitHub] [lucene-solr] jimczi commented on issue #854: Shared PQ Based Early Termination for Concurrent Search

2019-09-11 Thread GitBox
jimczi commented on issue #854: Shared PQ Based Early Termination for Concurrent Search URL: https://github.com/apache/lucene-solr/pull/854#issuecomment-530342313 > Is this proposal to continue collecting the full N per slice (thread work unit), until some/all of them have a full (top N) P

[GitHub] [lucene-solr] atris commented on issue #854: Shared PQ Based Early Termination for Concurrent Search

2019-09-11 Thread GitBox
atris commented on issue #854: Shared PQ Based Early Termination for Concurrent Search URL: https://github.com/apache/lucene-solr/pull/854#issuecomment-530343716 > > Is this proposal to continue collecting the full N per slice (thread work unit), until some/all of them have a full (top N)

[GitHub] [lucene-solr] jimczi commented on issue #854: Shared PQ Based Early Termination for Concurrent Search

2019-09-11 Thread GitBox
jimczi commented on issue #854: Shared PQ Based Early Termination for Concurrent Search URL: https://github.com/apache/lucene-solr/pull/854#issuecomment-530345711 I think that you need to ensure that the slice publishing the new min score is full, otherwise the min score cannot be the glob

[GitHub] [lucene-solr] atris commented on issue #854: Shared PQ Based Early Termination for Concurrent Search

2019-09-11 Thread GitBox
atris commented on issue #854: Shared PQ Based Early Termination for Concurrent Search URL: https://github.com/apache/lucene-solr/pull/854#issuecomment-530349367 > I think that you need to ensure that the slice publishing the new min score is full, otherwise the min score cannot be the glo

[GitHub] [lucene-solr] jimczi commented on issue #854: Shared PQ Based Early Termination for Concurrent Search

2019-09-11 Thread GitBox
jimczi commented on issue #854: Shared PQ Based Early Termination for Concurrent Search URL: https://github.com/apache/lucene-solr/pull/854#issuecomment-530373796 I am not sure that synchronization would be an issue in this model since the update of the global min score should be quick, we

[GitHub] [lucene-solr] atris commented on issue #854: Shared PQ Based Early Termination for Concurrent Search

2019-09-11 Thread GitBox
atris commented on issue #854: Shared PQ Based Early Termination for Concurrent Search URL: https://github.com/apache/lucene-solr/pull/854#issuecomment-530380308 > I am not sure that synchronization would be an issue in this model since the update of the global min score should be quick, w

[GitHub] [lucene-solr] jimczi commented on issue #854: Shared PQ Based Early Termination for Concurrent Search

2019-09-11 Thread GitBox
jimczi commented on issue #854: Shared PQ Based Early Termination for Concurrent Search URL: https://github.com/apache/lucene-solr/pull/854#issuecomment-530398602 > Thanks for offering. I have a skeletal PR in flight for this approach that I plan to publish tomorrow -- maybe we can iterate

[GitHub] [lucene-solr] msokolov merged pull request #862: LUCENE-8971: Enable constructing JapaneseTokenizer with custom dictio…

2019-09-11 Thread GitBox
msokolov merged pull request #862: LUCENE-8971: Enable constructing JapaneseTokenizer with custom dictio… URL: https://github.com/apache/lucene-solr/pull/862 This is an automated message from the Apache Git Service. To respo

[GitHub] [lucene-solr] atris commented on issue #854: Shared PQ Based Early Termination for Concurrent Search

2019-09-11 Thread GitBox
atris commented on issue #854: Shared PQ Based Early Termination for Concurrent Search URL: https://github.com/apache/lucene-solr/pull/854#issuecomment-530418576 > I think it would be simpler to keep the maximum minimum score on each slice. Each time a slice publish a new minimum score

[GitHub] [lucene-solr] msokolov commented on issue #862: LUCENE-8971: Enable constructing JapaneseTokenizer with custom dictio…

2019-09-11 Thread GitBox
msokolov commented on issue #862: LUCENE-8971: Enable constructing JapaneseTokenizer with custom dictio… URL: https://github.com/apache/lucene-solr/pull/862#issuecomment-530426989 Also back-ported to 8x branch - this was a clean cherry-pick, so I didn't post a PR.

[GitHub] [lucene-solr] jimczi commented on issue #854: Shared PQ Based Early Termination for Concurrent Search

2019-09-11 Thread GitBox
jimczi commented on issue #854: Shared PQ Based Early Termination for Concurrent Search URL: https://github.com/apache/lucene-solr/pull/854#issuecomment-530443453 > Which contract would that be? I believe that today, the contract that we call updateMinCompetitiveScore only when the PQ is f

[GitHub] [lucene-solr] jimczi commented on issue #862: LUCENE-8971: Enable constructing JapaneseTokenizer with custom dictio…

2019-09-11 Thread GitBox
jimczi commented on issue #862: LUCENE-8971: Enable constructing JapaneseTokenizer with custom dictio… URL: https://github.com/apache/lucene-solr/pull/862#issuecomment-530445644 Can we wait that we agreed on the full plan before backporting to 8x branch ? I don't see the full picture of t

[GitHub] [lucene-solr] atris commented on issue #854: Shared PQ Based Early Termination for Concurrent Search

2019-09-11 Thread GitBox
atris commented on issue #854: Shared PQ Based Early Termination for Concurrent Search URL: https://github.com/apache/lucene-solr/pull/854#issuecomment-530453877 > We need to ensure that the minimum score of the slice can be set as the maximum minimum score for all slices, however if t

[GitHub] [lucene-solr] msokolov commented on issue #862: LUCENE-8971: Enable constructing JapaneseTokenizer with custom dictio…

2019-09-11 Thread GitBox
msokolov commented on issue #862: LUCENE-8971: Enable constructing JapaneseTokenizer with custom dictio… URL: https://github.com/apache/lucene-solr/pull/862#issuecomment-530481447 Jim, I'm sorry - there was no comment for several days, so I assumed it was OK to push. Let's discuss, and if

[GitHub] [lucene-solr] msokolov commented on issue #862: LUCENE-8971: Enable constructing JapaneseTokenizer with custom dictio…

2019-09-11 Thread GitBox
msokolov commented on issue #862: LUCENE-8971: Enable constructing JapaneseTokenizer with custom dictio… URL: https://github.com/apache/lucene-solr/pull/862#issuecomment-530482206 But to answer your question a little more fully: the way this works is, you use DictionaryBuilder, in the same

[GitHub] [lucene-solr] jimczi commented on issue #862: LUCENE-8971: Enable constructing JapaneseTokenizer with custom dictio…

2019-09-11 Thread GitBox
jimczi commented on issue #862: LUCENE-8971: Enable constructing JapaneseTokenizer with custom dictio… URL: https://github.com/apache/lucene-solr/pull/862#issuecomment-530492308 > Jim, I'm sorry - there was no comment for several days, so I assumed it was OK to push No problem.

[GitHub] [lucene-solr] msokolov commented on issue #862: LUCENE-8971: Enable constructing JapaneseTokenizer with custom dictio…

2019-09-11 Thread GitBox
msokolov commented on issue #862: LUCENE-8971: Enable constructing JapaneseTokenizer with custom dictio… URL: https://github.com/apache/lucene-solr/pull/862#issuecomment-530494478 > Should it be marked experimental then ? The fact that we ship a single dictionary within the jar also ensure

[GitHub] [lucene-solr] jimczi commented on issue #854: Shared PQ Based Early Termination for Concurrent Search

2019-09-11 Thread GitBox
jimczi commented on issue #854: Shared PQ Based Early Termination for Concurrent Search URL: https://github.com/apache/lucene-solr/pull/854#issuecomment-530504547 I think we're talking of different approaches, hence the confusion. It is correct that we can start setting the minimum score w

[GitHub] [lucene-solr] atris commented on issue #854: Shared PQ Based Early Termination for Concurrent Search

2019-09-11 Thread GitBox
atris commented on issue #854: Shared PQ Based Early Termination for Concurrent Search URL: https://github.com/apache/lucene-solr/pull/854#issuecomment-530509226 > I think we're talking of different approaches, hence the confusion. It is correct that we can start setting the minimum score

[GitHub] [lucene-solr] jimczi commented on issue #854: Shared PQ Based Early Termination for Concurrent Search

2019-09-11 Thread GitBox
jimczi commented on issue #854: Shared PQ Based Early Termination for Concurrent Search URL: https://github.com/apache/lucene-solr/pull/854#issuecomment-530515005 > RE: synchronization, what are your thoughts on using a global shared array where each collector publishes its bottom value vs

[GitHub] [lucene-solr] mikemccand commented on issue #854: Shared PQ Based Early Termination for Concurrent Search

2019-09-11 Thread GitBox
mikemccand commented on issue #854: Shared PQ Based Early Termination for Concurrent Search URL: https://github.com/apache/lucene-solr/pull/854#issuecomment-530549495 I agree the "maximum minimum" score (for all local PQs that are full) is compelling as it'd require minimum synchronization

[GitHub] [lucene-solr] msokolov commented on issue #854: Shared PQ Based Early Termination for Concurrent Search

2019-09-11 Thread GitBox
msokolov commented on issue #854: Shared PQ Based Early Termination for Concurrent Search URL: https://github.com/apache/lucene-solr/pull/854#issuecomment-530575471 There is also the option of something in between. To avoid thread contention, could we just make uncoordinated read/write of

[GitHub] [lucene-solr] mocobeta commented on issue #862: LUCENE-8971: Enable constructing JapaneseTokenizer with custom dictio…

2019-09-11 Thread GitBox
mocobeta commented on issue #862: LUCENE-8971: Enable constructing JapaneseTokenizer with custom dictio… URL: https://github.com/apache/lucene-solr/pull/862#issuecomment-530614095 As far as LUCENE-8816 ("Decouple Kuromoji's morphological analyser and its dictionary") and related or sub iss

[GitHub] [lucene-solr] mocobeta edited a comment on issue #862: LUCENE-8971: Enable constructing JapaneseTokenizer with custom dictio…

2019-09-11 Thread GitBox
mocobeta edited a comment on issue #862: LUCENE-8971: Enable constructing JapaneseTokenizer with custom dictio… URL: https://github.com/apache/lucene-solr/pull/862#issuecomment-530614095 As far as LUCENE-8816 ("Decouple Kuromoji's morphological analyser and its dictionary") and related or

[GitHub] [lucene-solr] mocobeta commented on issue #862: LUCENE-8971: Enable constructing JapaneseTokenizer with custom dictio…

2019-09-11 Thread GitBox
mocobeta commented on issue #862: LUCENE-8971: Enable constructing JapaneseTokenizer with custom dictio… URL: https://github.com/apache/lucene-solr/pull/862#issuecomment-530624903 Just to be clear, to my understanding, this change has no effects on Solr or Elasticsearch built on Lucene 8.x

[GitHub] [lucene-solr] yipen9 opened a new pull request #875: merge master

2019-09-11 Thread GitBox
yipen9 opened a new pull request #875: merge master URL: https://github.com/apache/lucene-solr/pull/875 # Description Please provide a short description of the changes you're making with this pull request. # Solution Please provide a short description of the

[GitHub] [lucene-solr] atris commented on issue #854: Shared PQ Based Early Termination for Concurrent Search

2019-09-11 Thread GitBox
atris commented on issue #854: Shared PQ Based Early Termination for Concurrent Search URL: https://github.com/apache/lucene-solr/pull/854#issuecomment-530675406 > There is also the option of something in between. To avoid thread contention, could we just make uncoordinated read/write of a

[GitHub] [lucene-solr] iverase merged pull request #874: LUCENE-8976: Use exact distance between point and bounding rectangle in FloatPointNearestNeighbor

2019-09-11 Thread GitBox
iverase merged pull request #874: LUCENE-8976: Use exact distance between point and bounding rectangle in FloatPointNearestNeighbor URL: https://github.com/apache/lucene-solr/pull/874 This is an automated message from the Ap

[GitHub] [lucene-solr] atris commented on issue #854: Shared PQ Based Early Termination for Concurrent Search

2019-09-11 Thread GitBox
atris commented on issue #854: Shared PQ Based Early Termination for Concurrent Search URL: https://github.com/apache/lucene-solr/pull/854#issuecomment-530678743 > > RE: synchronization, what are your thoughts on using a global shared array where each collector publishes its bottom value v

[GitHub] [lucene-solr] thomaswoeckinger commented on issue #855: SOLR-13739: Improve performance on huge schema updates

2019-09-12 Thread GitBox
thomaswoeckinger commented on issue #855: SOLR-13739: Improve performance on huge schema updates URL: https://github.com/apache/lucene-solr/pull/855#issuecomment-530695879 @dsmiley ready to merge? This is an automated message

[GitHub] [lucene-solr] atris opened a new pull request #876: Use The Passed In Threshold Value in doConcurrentSearchWithThreshold

2019-09-12 Thread GitBox
atris opened a new pull request #876: Use The Passed In Threshold Value in doConcurrentSearchWithThreshold URL: https://github.com/apache/lucene-solr/pull/876 This is an automated message from the Apache Git Service. To resp

[GitHub] [lucene-solr] jimczi commented on issue #854: Shared PQ Based Early Termination for Concurrent Search

2019-09-12 Thread GitBox
jimczi commented on issue #854: Shared PQ Based Early Termination for Concurrent Search URL: https://github.com/apache/lucene-solr/pull/854#issuecomment-530794180 > I was thinking of a class similar to HitsThresholdChecker, allowing shared state, since, as you rightly said, the size of cri

[GitHub] [lucene-solr] atris opened a new pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-12 Thread GitBox
atris opened a new pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877 This commit introduces a mechanism for early termination where, for indices sorted by relevance, hits are collected in per thread queue.

[GitHub] [lucene-solr] atris commented on issue #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-12 Thread GitBox
atris commented on issue #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#issuecomment-530818867 cc @jimczi I have not implemented the logic for Paging collector yet -- can do it in a separate PR if this looks fine

[GitHub] [lucene-solr] ctargett commented on issue #843: SOLR-13714: Correct refguide regarding shardHandlerFactory solrconfig…

2019-09-12 Thread GitBox
ctargett commented on issue #843: SOLR-13714: Correct refguide regarding shardHandlerFactory solrconfig… URL: https://github.com/apache/lucene-solr/pull/843#issuecomment-530944197 +1, thanks for fixing this. This is an automa

[GitHub] [lucene-solr] ctargett merged pull request #843: SOLR-13714: Correct refguide regarding shardHandlerFactory solrconfig…

2019-09-12 Thread GitBox
ctargett merged pull request #843: SOLR-13714: Correct refguide regarding shardHandlerFactory solrconfig… URL: https://github.com/apache/lucene-solr/pull/843 This is an automated message from the Apache Git Service. To respo

[GitHub] [lucene-solr] eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud

2019-09-12 Thread GitBox
eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud URL: https://github.com/apache/lucene-solr/pull/864#discussion_r323984413 ## File path: solr/core/src/java/org/apache/solr/cloud/autoscaling/IndexSizeTrigger.java ## @@ -

[GitHub] [lucene-solr] eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud

2019-09-12 Thread GitBox
eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud URL: https://github.com/apache/lucene-solr/pull/864#discussion_r323981156 ## File path: solr/core/src/java/org/apache/solr/store/blob/process/CorePullTask.java ## @@ -0,0

[GitHub] [lucene-solr] eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud

2019-09-12 Thread GitBox
eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud URL: https://github.com/apache/lucene-solr/pull/864#discussion_r323974231 ## File path: solr/core/src/java/org/apache/solr/store/blob/provider/BlobStorageProvider.java ##

[GitHub] [lucene-solr] eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud

2019-09-12 Thread GitBox
eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud URL: https://github.com/apache/lucene-solr/pull/864#discussion_r323972376 ## File path: solr/core/src/java/org/apache/solr/store/blob/client/S3StorageClient.java ## @@ -0

[GitHub] [lucene-solr] eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud

2019-09-12 Thread GitBox
eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud URL: https://github.com/apache/lucene-solr/pull/864#discussion_r323974906 ## File path: solr/core/src/java/org/apache/solr/store/blob/provider/BlobStorageProvider.java ##

[GitHub] [lucene-solr] eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud

2019-09-12 Thread GitBox
eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud URL: https://github.com/apache/lucene-solr/pull/864#discussion_r323973484 ## File path: solr/core/src/java/org/apache/solr/store/blob/process/CorePullTracker.java ## @@ -

[GitHub] [lucene-solr] eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud

2019-09-12 Thread GitBox
eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud URL: https://github.com/apache/lucene-solr/pull/864#discussion_r323967328 ## File path: solr/core/src/java/org/apache/solr/handler/admin/RequestApplyUpdatesOp.java ## @@

[GitHub] [lucene-solr] eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud

2019-09-12 Thread GitBox
eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud URL: https://github.com/apache/lucene-solr/pull/864#discussion_r323982385 ## File path: solr/core/src/java/org/apache/solr/store/blob/process/CorePullTask.java ## @@ -0,0

[GitHub] [lucene-solr] eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud

2019-09-12 Thread GitBox
eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud URL: https://github.com/apache/lucene-solr/pull/864#discussion_r324023752 ## File path: solr/core/src/java/org/apache/solr/store/shared/SharedStoreManager.java ## @@ -0,0

[GitHub] [lucene-solr] eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud

2019-09-12 Thread GitBox
eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud URL: https://github.com/apache/lucene-solr/pull/864#discussion_r324026158 ## File path: solr/core/src/java/org/apache/solr/store/blob/client/BlobCoreMetadata.java ## @@ -

[GitHub] [lucene-solr] eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud

2019-09-12 Thread GitBox
eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud URL: https://github.com/apache/lucene-solr/pull/864#discussion_r323982385 ## File path: solr/core/src/java/org/apache/solr/store/blob/process/CorePullTask.java ## @@ -0,0

[GitHub] [lucene-solr] eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud

2019-09-12 Thread GitBox
eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud URL: https://github.com/apache/lucene-solr/pull/864#discussion_r323982385 ## File path: solr/core/src/java/org/apache/solr/store/blob/process/CorePullTask.java ## @@ -0,0

[GitHub] [lucene-solr] eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud

2019-09-12 Thread GitBox
eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud URL: https://github.com/apache/lucene-solr/pull/864#discussion_r324027378 ## File path: solr/core/src/java/org/apache/solr/store/blob/client/BlobCoreMetadataBuilder.java #

[GitHub] [lucene-solr] eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud

2019-09-12 Thread GitBox
eribeiro commented on a change in pull request #864: SOLR-13101 : Shared storage support in SolrCloud URL: https://github.com/apache/lucene-solr/pull/864#discussion_r324030543 ## File path: solr/core/src/java/org/apache/solr/store/blob/client/LocalStorageClient.java ## @@

[GitHub] [lucene-solr] atris commented on issue #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
atris commented on issue #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#issuecomment-531142523 I ran luceneutil with concurrent mode and wikimedium2m -- no degradation to QPS and some tasks show a reduction of around 30%

[GitHub] [lucene-solr] jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324097168 ## File path: lucene/core/src/java/org/apache/lucene/search/BottomValueChecker.java ## @@

[GitHub] [lucene-solr] jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324098325 ## File path: lucene/core/src/java/org/apache/lucene/search/TopScoreDocCollector.java ##

[GitHub] [lucene-solr] jpountz commented on a change in pull request #868: LUCENE-8975: Code Cleanup: Use entryset for map iteration wherever possible.

2019-09-13 Thread GitBox
jpountz commented on a change in pull request #868: LUCENE-8975: Code Cleanup: Use entryset for map iteration wherever possible. URL: https://github.com/apache/lucene-solr/pull/868#discussion_r324101947 ## File path: lucene/CHANGES.txt ## @@ -65,6 +65,8 @@ Other * LUCEN

[GitHub] [lucene-solr] atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324103214 ## File path: lucene/core/src/java/org/apache/lucene/search/TopScoreDocCollector.java ## @

[GitHub] [lucene-solr] atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324103471 ## File path: lucene/core/src/java/org/apache/lucene/search/BottomValueChecker.java ## @@

[GitHub] [lucene-solr] jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324105474 ## File path: lucene/core/src/java/org/apache/lucene/search/TopScoreDocCollector.java ##

[GitHub] [lucene-solr] jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324105474 ## File path: lucene/core/src/java/org/apache/lucene/search/TopScoreDocCollector.java ##

[GitHub] [lucene-solr] atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324116784 ## File path: lucene/core/src/java/org/apache/lucene/search/TopScoreDocCollector.java ## @

[GitHub] [lucene-solr] atris commented on issue #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
atris commented on issue #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#issuecomment-531174857 @jimczi Thanks for the comments -- updated the PR. Please let me know your thoughts -

[GitHub] [lucene-solr] jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324130644 ## File path: lucene/core/src/java/org/apache/lucene/search/BottomValueChecker.java ## @@

[GitHub] [lucene-solr] jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324130957 ## File path: lucene/core/src/java/org/apache/lucene/search/TopScoreDocCollector.java ##

[GitHub] [lucene-solr] jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324129841 ## File path: lucene/core/src/java/org/apache/lucene/search/TopScoreDocCollector.java ##

[GitHub] [lucene-solr] jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324130344 ## File path: lucene/core/src/java/org/apache/lucene/search/BottomValueChecker.java ## @@

[GitHub] [lucene-solr] jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324130518 ## File path: lucene/core/src/java/org/apache/lucene/search/BottomValueChecker.java ## @@

[GitHub] [lucene-solr] jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324125480 ## File path: lucene/core/src/java/org/apache/lucene/search/TopScoreDocCollector.java ##

[GitHub] [lucene-solr] jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324122789 ## File path: lucene/core/src/java/org/apache/lucene/search/BottomValueChecker.java ## @@

[GitHub] [lucene-solr] KoenDG commented on a change in pull request #868: LUCENE-8975: Code Cleanup: Use entryset for map iteration wherever possible.

2019-09-13 Thread GitBox
KoenDG commented on a change in pull request #868: LUCENE-8975: Code Cleanup: Use entryset for map iteration wherever possible. URL: https://github.com/apache/lucene-solr/pull/868#discussion_r324133535 ## File path: lucene/CHANGES.txt ## @@ -65,6 +65,8 @@ Other * LUCENE

[GitHub] [lucene-solr] atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324141301 ## File path: lucene/core/src/java/org/apache/lucene/search/TopScoreDocCollector.java ## @

[GitHub] [lucene-solr] atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324141173 ## File path: lucene/core/src/java/org/apache/lucene/search/TopScoreDocCollector.java ## @

[GitHub] [lucene-solr] atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324145887 ## File path: lucene/core/src/java/org/apache/lucene/search/BottomValueChecker.java ## @@

[GitHub] [lucene-solr] iverase commented on issue #770: LUCENE-8746: Component2D topology library that works on encoded space

2019-09-13 Thread GitBox
iverase commented on issue #770: LUCENE-8746: Component2D topology library that works on encoded space URL: https://github.com/apache/lucene-solr/pull/770#issuecomment-531219042 Closing this PR. I am opening a new one with narrow scope --

[GitHub] [lucene-solr] iverase closed pull request #770: LUCENE-8746: Component2D topology library that works on encoded space

2019-09-13 Thread GitBox
iverase closed pull request #770: LUCENE-8746: Component2D topology library that works on encoded space URL: https://github.com/apache/lucene-solr/pull/770 This is an automated message from the Apache Git Service. To respond

[GitHub] [lucene-solr] jpountz commented on a change in pull request #865: LUCENE-8973: XYRectangle2D should work on float space

2019-09-13 Thread GitBox
jpountz commented on a change in pull request #865: LUCENE-8973: XYRectangle2D should work on float space URL: https://github.com/apache/lucene-solr/pull/865#discussion_r324172772 ## File path: lucene/sandbox/src/java/org/apache/lucene/geo/XYRectangle2D.java ## @@ -16,42 +

[GitHub] [lucene-solr] jpountz commented on issue #875: merge master

2019-09-13 Thread GitBox
jpountz commented on issue #875: merge master URL: https://github.com/apache/lucene-solr/pull/875#issuecomment-531221639 Probably open by mistake. This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] [lucene-solr] jpountz closed pull request #875: merge master

2019-09-13 Thread GitBox
jpountz closed pull request #875: merge master URL: https://github.com/apache/lucene-solr/pull/875 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

[GitHub] [lucene-solr] atris commented on issue #815: LUCENE-8213: Introduce Asynchronous Caching in LRUQueryCache

2019-09-13 Thread GitBox
atris commented on issue #815: LUCENE-8213: Introduce Asynchronous Caching in LRUQueryCache URL: https://github.com/apache/lucene-solr/pull/815#issuecomment-531221868 Any further thoughts on this one? This is an automated mes

[GitHub] [lucene-solr] jpountz merged pull request #868: LUCENE-8975: Code Cleanup: Use entryset for map iteration wherever possible.

2019-09-13 Thread GitBox
jpountz merged pull request #868: LUCENE-8975: Code Cleanup: Use entryset for map iteration wherever possible. URL: https://github.com/apache/lucene-solr/pull/868 This is an automated message from the Apache Git Service. To

[GitHub] [lucene-solr] atris commented on issue #876: Use The Passed In Threshold Value in doConcurrentSearchWithThreshold

2019-09-13 Thread GitBox
atris commented on issue #876: Use The Passed In Threshold Value in doConcurrentSearchWithThreshold URL: https://github.com/apache/lucene-solr/pull/876#issuecomment-531226956 Can we merge this one? This is a simple fix to an existing test method. ---

[GitHub] [lucene-solr] atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324162028 ## File path: lucene/core/src/java/org/apache/lucene/search/TopScoreDocCollector.java ## @

[GitHub] [lucene-solr] atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324161934 ## File path: lucene/core/src/java/org/apache/lucene/search/BottomValueChecker.java ## @@

[GitHub] [lucene-solr] atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324146082 ## File path: lucene/core/src/java/org/apache/lucene/search/BottomValueChecker.java ## @@

[GitHub] [lucene-solr] atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324146126 ## File path: lucene/core/src/java/org/apache/lucene/search/BottomValueChecker.java ## @@

[GitHub] [lucene-solr] atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324181290 ## File path: lucene/core/src/java/org/apache/lucene/search/BottomValueChecker.java ## @@

[GitHub] [lucene-solr] atris commented on issue #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
atris commented on issue #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#issuecomment-531228068 @jimczi Updated, please see. One thing we could do is cache the bottom docID (doc + docBase) in BottomValueChecker, but

[GitHub] [lucene-solr] msokolov commented on issue #876: Use The Passed In Threshold Value in doConcurrentSearchWithThreshold

2019-09-13 Thread GitBox
msokolov commented on issue #876: Use The Passed In Threshold Value in doConcurrentSearchWithThreshold URL: https://github.com/apache/lucene-solr/pull/876#issuecomment-531229416 I'll commit it. As I looked at this I couldn't help noticing a little spelling nit; we have {{thresHold}} but "t

[GitHub] [lucene-solr] atris commented on issue #876: Use The Passed In Threshold Value in doConcurrentSearchWithThreshold

2019-09-13 Thread GitBox
atris commented on issue #876: Use The Passed In Threshold Value in doConcurrentSearchWithThreshold URL: https://github.com/apache/lucene-solr/pull/876#issuecomment-531230646 Thanks, missed the camel casing, fixed now This is

[GitHub] [lucene-solr] msokolov merged pull request #876: Use The Passed In Threshold Value in doConcurrentSearchWithThreshold

2019-09-13 Thread GitBox
msokolov merged pull request #876: Use The Passed In Threshold Value in doConcurrentSearchWithThreshold URL: https://github.com/apache/lucene-solr/pull/876 This is an automated message from the Apache Git Service. To respond

[GitHub] [lucene-solr] msokolov commented on issue #876: Use The Passed In Threshold Value in doConcurrentSearchWithThreshold

2019-09-13 Thread GitBox
msokolov commented on issue #876: Use The Passed In Threshold Value in doConcurrentSearchWithThreshold URL: https://github.com/apache/lucene-solr/pull/876#issuecomment-531233209 thanks! merged This is an automated message fro

[GitHub] [lucene-solr] atris commented on issue #876: Use The Passed In Threshold Value in doConcurrentSearchWithThreshold

2019-09-13 Thread GitBox
atris commented on issue #876: Use The Passed In Threshold Value in doConcurrentSearchWithThreshold URL: https://github.com/apache/lucene-solr/pull/876#issuecomment-531233493 Thanks! This is an automated message from the Apac

[GitHub] [lucene-solr] jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324216889 ## File path: lucene/core/src/java/org/apache/lucene/search/BottomValueChecker.java ## @@

[GitHub] [lucene-solr] jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324218620 ## File path: lucene/core/src/java/org/apache/lucene/search/TopScoreDocCollector.java ##

[GitHub] [lucene-solr] jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
jimczi commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324218422 ## File path: lucene/core/src/java/org/apache/lucene/search/TopScoreDocCollector.java ##

[GitHub] [lucene-solr] atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
atris commented on a change in pull request #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#discussion_r324221245 ## File path: lucene/core/src/java/org/apache/lucene/search/TopScoreDocCollector.java ## @

[GitHub] [lucene-solr] atris commented on issue #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
atris commented on issue #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#issuecomment-531261223 @jimczi Updated This is an automated message from the Apache

[GitHub] [lucene-solr] jimczi commented on issue #877: LUCENE-8978: Maximal Bottom Score Based Early Termination

2019-09-13 Thread GitBox
jimczi commented on issue #877: LUCENE-8978: Maximal Bottom Score Based Early Termination URL: https://github.com/apache/lucene-solr/pull/877#issuecomment-531266829 Can you run another benchmark now that we propagate the global min score to the scorer ? ---

<    1   2   3   4   5   6   7   8   9   10   >