jainankitk commented on code in PR #14439:
URL: https://github.com/apache/lucene/pull/14439#discussion_r2050147491


##########
lucene/sandbox/src/test/org/apache/lucene/sandbox/facet/plain/histograms/TestHistogramCollectorManager.java:
##########
@@ -115,6 +117,38 @@ public void testSkipIndexWithSortAndLowInterval() throws 
IOException {
             .setCodec(TestUtil.alwaysDocValuesFormat(new 
Lucene90DocValuesFormat(3))));
   }
 
+  public void testMultiRangePointTreeCollector() throws IOException {

Review Comment:
   Okay, I have added logic for collecting only for one of the partitions if 
intra-segment concurrency is enabled. Ran the test 50k times after that without 
any failures:
   
   ```
   > Task :lucene:core:testClasses
   > Task :lucene:core:test
   > Task :lucene:core:wipeTaskTemp
   
   > Task :lucene:sandbox:test
   :lucene:sandbox:test (SUCCESS): 50000 test(s)
   
   > Task :lucene:sandbox:wipeTaskTemp
   The slowest suites (exceeding 1s) during this run:
     580.41s TestHistogramCollectorManager (:lucene:sandbox)
   
   BUILD SUCCESSFUL in 9m 58s
   246 actionable tasks: 181 executed, 65 up-to-date
   ```



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