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

Reply via email to