Hi Chris, I can't find the PR.
I am interested, because I wrote the original ParallelReader tests. IMHO the parallel readers are so sensitive to random changes, the test setup should not use any indexwriter randomization at all. ParallelReader is also seldomly used, maybe we should remove support at some point. I don't know anybody using it, because it is very complicated to maintain consistent indexes. It only works with stable merge policies. Uwe Am 2. Dezember 2023 09:34:46 MEZ schrieb Chris Hegarty <[email protected]>: >Hi, > >I noticed this failure locally, and opened a PR for it yesterday. It is a >test issues, and indeed related to the recent merge policy test >randomization change. > >-Chris > >On Saturday, December 2, 2023, Patrick Zhai <[email protected]> wrote: > >> Seems it's because this MockRandomMergePolicy change >> <https://github.com/apache/lucene/blob/main/lucene/test-framework/src/java/org/apache/lucene/tests/index/MockRandomMergePolicy.java#L242> >> recently >> makes ParallelLeafReader unhappy - it's reading two parallel segments from >> 2 dir and this MP makes one of the segments' documents order reversed. >> >> But should be just test util issue and not affecting release. >> >> Adrien do you want to take a look? I'm not sure what's the best way to fix >> it, adding an index sort for that test seems a bit overkill? >> >> Patrick >> >> On Fri, Dec 1, 2023 at 2:06 PM Michael McCandless < >> [email protected]> wrote: >> >>> Hmm this reproduces for me, and looks new/unique. Could it be related to >>> recent 9.9.0 changes / release blocker? >>> >>> Mike >>> >>> On Fri, Dec 1, 2023 at 3:33 PM Policeman Jenkins Server < >>> [email protected]> wrote: >>> >>>> Build: https://jenkins.thetaphi.de/job/Lucene-9.x-Linux/14204/ >>>> Java: 64bit/hotspot/jdk-11.0.21 -XX:+UseCompressedOops -XX:+UseParallelGC >>>> >>>> 1 tests failed. >>>> FAILED: org.apache.lucene.index.TestParallelLeafReader.testQueries >>>> >>>> Error Message: >>>> org.junit.ComparisonFailure: expected:<v[2]> but was:<v[1]> >>>> >>>> Stack Trace: >>>> org.junit.ComparisonFailure: expected:<v[2]> but was:<v[1]> >>>> at __randomizedtesting.SeedInfo.seed([6CA57EA3FB50CA0D: >>>> 302BB278E1397FA3]:0) >>>> at org.junit.Assert.assertEquals(Assert.java:117) >>>> at org.junit.Assert.assertEquals(Assert.java:146) >>>> at org.apache.lucene.index.TestParallelLeafReader.queryTest( >>>> TestParallelLeafReader.java:263) >>>> at org.apache.lucene.index.TestParallelLeafReader.testQueries( >>>> TestParallelLeafReader.java:48) >>>> at >>>> java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native >>>> Method) >>>> at java.base/jdk.internal.reflect.NativeMethodAccessorImpl. >>>> invoke(NativeMethodAccessorImpl.java:62) >>>> at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl. >>>> invoke(DelegatingMethodAccessorImpl.java:43) >>>> at java.base/java.lang.reflect.Method.invoke(Method.java:566) >>>> at com.carrotsearch.randomizedtesting.RandomizedRunner.invoke( >>>> RandomizedRunner.java:1758) >>>> at com.carrotsearch.randomizedtesting. >>>> RandomizedRunner$8.evaluate(RandomizedRunner.java:946) >>>> at com.carrotsearch.randomizedtesting. >>>> RandomizedRunner$9.evaluate(RandomizedRunner.java:982) >>>> at com.carrotsearch.randomizedtesting. >>>> RandomizedRunner$10.evaluate(RandomizedRunner.java:996) >>>> at org.apache.lucene.tests.util.TestRuleSetupTeardownChained$ >>>> 1.evaluate(TestRuleSetupTeardownChained.java:48) >>>> at org.apache.lucene.tests.util.AbstractBeforeAfterRule$1. >>>> evaluate(AbstractBeforeAfterRule.java:43) >>>> at org.apache.lucene.tests.util.TestRuleThreadAndTestName$1. >>>> evaluate(TestRuleThreadAndTestName.java:45) >>>> at org.apache.lucene.tests.util.TestRuleIgnoreAfterMaxFailures >>>> $1.evaluate(TestRuleIgnoreAfterMaxFailures.java:60) >>>> at org.apache.lucene.tests.util.TestRuleMarkFailure$1. >>>> evaluate(TestRuleMarkFailure.java:44) >>>> at org.junit.rules.RunRules.evaluate(RunRules.java:20) >>>> at com.carrotsearch.randomizedtesting.rules. >>>> StatementAdapter.evaluate(StatementAdapter.java:36) >>>> at com.carrotsearch.randomizedtesting.ThreadLeakControl$ >>>> StatementRunner.run(ThreadLeakControl.java:390) >>>> at com.carrotsearch.randomizedtesting.ThreadLeakControl. >>>> forkTimeoutingTask(ThreadLeakControl.java:843) >>>> at com.carrotsearch.randomizedtesting. >>>> ThreadLeakControl$3.evaluate(ThreadLeakControl.java:490) >>>> at com.carrotsearch.randomizedtesting.RandomizedRunner. >>>> runSingleTest(RandomizedRunner.java:955) >>>> at com.carrotsearch.randomizedtesting. >>>> RandomizedRunner$5.evaluate(RandomizedRunner.java:840) >>>> at com.carrotsearch.randomizedtesting. >>>> RandomizedRunner$6.evaluate(RandomizedRunner.java:891) >>>> at com.carrotsearch.randomizedtesting. >>>> RandomizedRunner$7.evaluate(RandomizedRunner.java:902) >>>> at org.apache.lucene.tests.util.AbstractBeforeAfterRule$1. >>>> evaluate(AbstractBeforeAfterRule.java:43) >>>> at com.carrotsearch.randomizedtesting.rules. >>>> StatementAdapter.evaluate(StatementAdapter.java:36) >>>> at org.apache.lucene.tests.util.TestRuleStoreClassName$1. >>>> evaluate(TestRuleStoreClassName.java:38) >>>> at com.carrotsearch.randomizedtesting.rules. >>>> NoShadowingOrOverridesOnMethodsRule$1.evaluate( >>>> NoShadowingOrOverridesOnMethodsRule.java:40) >>>> at com.carrotsearch.randomizedtesting.rules. >>>> NoShadowingOrOverridesOnMethodsRule$1.evaluate( >>>> NoShadowingOrOverridesOnMethodsRule.java:40) >>>> at com.carrotsearch.randomizedtesting.rules. >>>> StatementAdapter.evaluate(StatementAdapter.java:36) >>>> at com.carrotsearch.randomizedtesting.rules. >>>> StatementAdapter.evaluate(StatementAdapter.java:36) >>>> at org.apache.lucene.tests.util.TestRuleAssertionsRequired$1. >>>> evaluate(TestRuleAssertionsRequired.java:53) >>>> at org.apache.lucene.tests.util.AbstractBeforeAfterRule$1. >>>> evaluate(AbstractBeforeAfterRule.java:43) >>>> at org.apache.lucene.tests.util.TestRuleMarkFailure$1. >>>> evaluate(TestRuleMarkFailure.java:44) >>>> at org.apache.lucene.tests.util.TestRuleIgnoreAfterMaxFailures >>>> $1.evaluate(TestRuleIgnoreAfterMaxFailures.java:60) >>>> at org.apache.lucene.tests.util.TestRuleIgnoreTestSuites$1. >>>> evaluate(TestRuleIgnoreTestSuites.java:47) >>>> at org.junit.rules.RunRules.evaluate(RunRules.java:20) >>>> at com.carrotsearch.randomizedtesting.rules. >>>> StatementAdapter.evaluate(StatementAdapter.java:36) >>>> at com.carrotsearch.randomizedtesting.ThreadLeakControl$ >>>> StatementRunner.run(ThreadLeakControl.java:390) >>>> at com.carrotsearch.randomizedtesting.ThreadLeakControl.lambda$ >>>> forkTimeoutingTask$0(ThreadLeakControl.java:850) >>>> at java.base/java.lang.Thread.run(Thread.java:829) >>>> >>>> --------------------------------------------------------------------- >>>> To unsubscribe, e-mail: [email protected] >>>> For additional commands, e-mail: [email protected] >>> >>> -- Uwe Schindler Achterdiek 19, 28357 Bremen https://www.thetaphi.de
