gsmiller commented on PR #14238:
URL: https://github.com/apache/lucene/pull/14238#issuecomment-2666152482

   @houserjohn it looks like the randomized testing failed in some recent runs 
(see: 
[here](https://github.com/apache/lucene/actions/runs/13388526608/job/37390674327)):
   ```
   TestDynamicRangeUtil > testComputeDynamicNumericRangesWithRandomValues FAILED
       java.lang.AssertionError
           at 
__randomizedtesting.SeedInfo.seed([50D292E371306B1:9BCACADABDA87D35]:0)
           at junit@4.13.1/org.junit.Assert.fail(Assert.java:87)
           at junit@4.13.1/org.junit.Assert.assertTrue(Assert.java:42)
           at junit@4.13.1/org.junit.Assert.assertTrue(Assert.java:53)
   ...
     2> NOTE: reproduce with: gradlew test --tests 
TestDynamicRangeUtil.testComputeDynamicNumericRangesWithRandomValues 
-Dtests.seed=50D292E371306B1 -Dtests.locale=kl-Latn-GL 
-Dtests.timezone=America/Mendoza -Dtests.asserts=true 
-Dtests.file.encoding=UTF-8
     2> NOTE: test params are: codec=Asserting(Lucene101): {}, docValues:{}, 
maxPointsInLeafNode=1816, maxMBSortInHeap=7.672327997432254, 
sim=Asserting(RandomSimilarity(queryNorm=false): {}), locale=kl-Latn-GL, 
timezone=America/Mendoza
     2> NOTE: Mac OS X 14.7.2 aarch64/Eclipse Adoptium 21.0.5 
(64-bit)/cpus=3,threads=1,free=190300384,total=268435456
     2> NOTE: All tests run in this JVM: [TestDrillDownQuery, 
TestDrillSideways, TestFacetQuery, TestFacetUtils, TestFacetsConfig, 
TestLongValueFacetCounts, TestMultiDoubleValuesSource, 
TestMultiLongValuesSource, TestMultipleIndexFields, TestParallelDrillSideways, 
TestRandomSamplingFacetsCollector, TestStringValueFacetCounts, 
TestExactFacetSetMatcher, TestMatchingFacetSetsCounts, 
TestRangeFacetSetMatcher, TestDynamicRangeUtil]
   ```
   
   (thanks @stefanvodita for catching this!)
   
   The good news is this reproduced for me locally with the seed from the 
linked failure:
   `./gradlew test --tests 
TestDynamicRangeUtil.testComputeDynamicNumericRangesWithRandomValues 
-Dtests.seed=50D292E371306B1 -Dtests.locale=kl-Latn-GL 
-Dtests.timezone=America/Mendoza -Dtests.asserts=true 
-Dtests.file.encoding=UTF-8`
   
   @stefanvodita reverted this change and the corresponding back port already 
(see #14252 and #14253). So I think the right next steps here are to figure out 
what's causing the test failure (did randomization catch an actual bug in the 
implementation, or is there an invalid test assertion somewhere?), then reopen 
a new PR with this change that includes the fix. Thanks!


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