Hi, > On 24 Sep 2024, at 09:37, Uwe Schindler <u...@thetaphi.de> wrote: > > At least one run with all JDKs including JDK 23 succeeded after approx 4 hrs: > https://jenkins.thetaphi.de/job/Lucene-Release-Tester/36/console > Tested were: > Java 11 JAVA_HOME=/home/jenkins/tools/java/64bit/hotspot/latest-jdk11 > Java 17 JAVA_HOME=/home/jenkins/tools/java/64bit/hotspot/latest-jdk17 > Java 19 JAVA_HOME=/home/jenkins/tools/java/64bit/hotspot/latest-jdk19 > Java 20 JAVA_HOME=/home/jenkins/tools/java/64bit/hotspot/latest-jdk20 > Java 21 JAVA_HOME=/home/jenkins/tools/java/64bit/hotspot/latest-jdk21 > Java 22 JAVA_HOME=/home/jenkins/tools/java/64bit/hotspot/latest-jdk22 > Java 23 JAVA_HOME=/home/jenkins/tools/java/64bit/hotspot/latest-jdk23 > > SUCCESS! [3:58:29.742469] > > Finished: SUCCESS > > Looks like the general issues are gone, if we should or must fix the test is > up to the maintainers of the merge policy. If the issue does not prevent the > code from working correctly and is just a test issue, I'd +1 for release. > +1 for release, unless the uncovered issue with TieredMergePolicy causes > malfunction.
My analysis concludes that there is no product bug here. Yeah, maybe the policy could be a little more aggressive in its merging, but the issue that the test runs into looks like a corner case on the boundary of what the heuristics look for. https://github.com/apache/lucene/issues/13818#issuecomment-2371868712 It would be good to have Adrien or Mike confirm. -Chris. > Uwe > Am 23.09.2024 um 22:28 schrieb Chris Hegarty: >> Thanks Uwe, >> >> I reproduced this failure on main and branch_9_12. >> >> The following issue has been created to track this, >> https://github.com/apache/lucene/issues/13818. >> >> Git bisect has identified the first failure as "Add target search >> concurrency to TieredMergePolicy (#13430)" >> >> -Chris. >> >> >>> On 23 Sep 2024, at 18:10, Uwe Schindler <u...@thetaphi.de> wrote: >>> >>> Unfortunately, the first Smoketester run failed with a test failure (in >>> Java 19): >>> >>> TestTieredMergePolicy > testSimulateAppendOnly FAILED >>> java.lang.AssertionError: mergeFactor=4 minSegmentBytes=38,046,720 >>> maxMergedSegmentBytes=5,213,519,872 segmentsPerTier=4.00000 >>> maxMergeAtOnce=24 numSegments=43 allowed=41.0000 totalBytes=8,132,505,600 >>> delPercentage=0.00000 deletesPctAllowed=39.9912 targetNumSegments=32 >>> >>> NOTE: reproduce with: gradlew test --tests >>> TestTieredMergePolicy.testSimulateAppendOnly -Dtests.seed=B90724898E61B17C >>> -Dtests.multiplier=1 -Dtests.nightly=true -Dtests.locale=bm >>> -Dtests.timezone=America/Metlakatla -Dtests.asserts=true >>> -Dtests.file.encoding=UTF-8 >>> >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org >> For additional commands, e-mail: dev-h...@lucene.apache.org >> >> > -- > Uwe Schindler > Achterdiek 19, D-28357 Bremen > https://www.thetaphi.de > eMail: u...@thetaphi.de --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org