Skip to content

Conversation

@lhotari
Copy link
Member

@lhotari lhotari commented May 6, 2025

Motivation

In Pulsar CI, Gradle Develocity Maven Extension is used to push build scans to ASF's Develocity server.

Current Gradle Develocity Maven Extension fails on Java 24 with this type of exception:

[ERROR] Error executing a DevelocityListener callback
org.codehaus.groovy.GroovyBugError: BUG! exception in phase 'semantic analysis' in source unit '/Users/lari/workspace-pulsar/pulsar/.mvn/develocity-custom-user-data.groovy' Unsupported class file major version 68
    at org.codehaus.groovy.control.CompilationUnit$ISourceUnitOperation.doPhaseOperation (CompilationUnit.java:906)
    at org.codehaus.groovy.control.CompilationUnit.compile (CompilationUnit.java:628)
    at groovy.lang.GroovyClassLoader.doParseClass (GroovyClassLoader.java:389)
    at groovy.lang.GroovyClassLoader.lambda$parseClass$3 (GroovyClassLoader.java:332)
    at org.codehaus.groovy.runtime.memoize.StampedCommonCache.compute (StampedCommonCache.java:163)
    at org.codehaus.groovy.runtime.memoize.StampedCommonCache.getAndPut (StampedCommonCache.java:154)
    at groovy.lang.GroovyClassLoader.parseClass (GroovyClassLoader.java:330)
    at groovy.lang.GroovyShell.parseClass (GroovyShell.java:526)
    at groovy.lang.GroovyShell.parse (GroovyShell.java:538)
    at groovy.lang.GroovyShell.evaluate (GroovyShell.java:426)
    at groovy.lang.GroovyShell.evaluate (GroovyShell.java:470)
    at com.gradle.GroovyScriptUserData.evaluateGroovyScript (GroovyScriptUserData.java:41)
    at com.gradle.GroovyScriptUserData.evaluate (GroovyScriptUserData.java:18)
    at com.gradle.CommonCustomUserDataListener.configure (CommonCustomUserDataListener.java:32)
    at com.gradle.CommonCustomUserDataDevelocityListener.configure (CommonCustomUserDataDevelocityListener.java:20)
    at com.gradle.maven.internal.DevelocityLifecycleManager.a (SourceFile:400)
    at java.lang.Iterable.forEach (Iterable.java:75)
    at java.util.Collections$UnmodifiableCollection.forEach (Collections.java:1117)
    at com.gradle.maven.internal.DevelocityLifecycleManager.b (SourceFile:398)
    at com.gradle.maven.internal.DevelocityLifecycleManager.c (SourceFile:370)
    at java.util.stream.ForEachOps$ForEachOp$OfRef.accept (ForEachOps.java:186)
    at java.util.stream.ReferencePipeline$15$1.accept (ReferencePipeline.java:581)
    at java.util.stream.IntPipeline$1$1.accept (IntPipeline.java:180)
    at java.util.stream.Streams$RangeIntSpliterator.forEachRemaining (Streams.java:108)
    at java.util.Spliterator$OfInt.forEachRemaining (Spliterator.java:712)
    at java.util.stream.AbstractPipeline.copyInto (AbstractPipeline.java:570)
    at java.util.stream.AbstractPipeline.wrapAndCopyInto (AbstractPipeline.java:560)
    at java.util.stream.ForEachOps$ForEachOp.evaluateSequential (ForEachOps.java:153)
    at java.util.stream.ForEachOps$ForEachOp$OfRef.evaluateSequential (ForEachOps.java:176)
    at java.util.stream.AbstractPipeline.evaluate (AbstractPipeline.java:265)
    at java.util.stream.ReferencePipeline.forEach (ReferencePipeline.java:636)
    at com.gradle.maven.internal.DevelocityLifecycleManager.a (SourceFile:370)
    at com.gradle.maven.internal.DevelocityLifecycleManager.onEvent (SourceFile:305)
    at org.apache.maven.eventspy.internal.EventSpyDispatcher.onEvent (EventSpyDispatcher.java:86)
    at org.apache.maven.eventspy.internal.EventSpyExecutionListener.sessionStarted (EventSpyExecutionListener.java:48)
    at org.apache.maven.lifecycle.internal.DefaultExecutionEventCatapult.fire (DefaultExecutionEventCatapult.java:54)
    at org.apache.maven.lifecycle.internal.DefaultExecutionEventCatapult.fire (DefaultExecutionEventCatapult.java:38)
    at org.apache.maven.lifecycle.internal.LifecycleStarter.execute (LifecycleStarter.java:71)
    at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:261)
    at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:173)
    at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:101)
    at org.apache.maven.cli.MavenCli.execute (MavenCli.java:906)
    at org.apache.maven.cli.MavenCli.doMain (MavenCli.java:283)
    at org.apache.maven.cli.MavenCli.main (MavenCli.java:206)
    at jdk.internal.reflect.DirectMethodHandleAccessor.invoke (DirectMethodHandleAccessor.java:104)
    at java.lang.reflect.Method.invoke (Method.java:565)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced (Launcher.java:255)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launch (Launcher.java:201)
    at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode (Launcher.java:361)
    at org.codehaus.plexus.classworlds.launcher.Launcher.main (Launcher.java:314)
Caused by: java.lang.IllegalArgumentException: Unsupported class file major version 68

Modifications

  • upgrade com.gradle:common-custom-user-data-maven-extension to 2.0.2 version which includes a fix for JDK 24
  • upgrade com.gradle:develocity-maven-extension to 1.23.2

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

@lhotari lhotari added this to the 4.1.0 milestone May 6, 2025
@lhotari lhotari requested review from dao-jun, heesung-sohn and nodece May 6, 2025 16:58
@lhotari lhotari self-assigned this May 6, 2025
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label May 6, 2025
@codecov-commenter
Copy link

codecov-commenter commented May 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.20%. Comparing base (bbc6224) to head (e2eb9c2).
Report is 1077 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #24260      +/-   ##
============================================
+ Coverage     73.57%   74.20%   +0.63%     
+ Complexity    32624    32191     -433     
============================================
  Files          1877     1866      -11     
  Lines        139502   144968    +5466     
  Branches      15299    16561    +1262     
============================================
+ Hits         102638   107576    +4938     
+ Misses        28908    28870      -38     
- Partials       7956     8522     +566     
Flag Coverage Δ
inttests 26.70% <ø> (+2.12%) ⬆️
systests 23.33% <ø> (-0.99%) ⬇️
unittests 73.68% <ø> (+0.83%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1079 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@lhotari lhotari merged commit 0350a76 into apache:master May 6, 2025
54 checks passed
lhotari added a commit that referenced this pull request May 9, 2025
lhotari added a commit that referenced this pull request May 9, 2025
lhotari added a commit that referenced this pull request May 9, 2025
manas-ctds pushed a commit to datastax/pulsar that referenced this pull request May 14, 2025
…es (apache#24260)

(cherry picked from commit 0350a76)
(cherry picked from commit b992578)
manas-ctds pushed a commit to datastax/pulsar that referenced this pull request May 14, 2025
…es (apache#24260)

(cherry picked from commit 0350a76)
(cherry picked from commit cfe7ac5)
manas-ctds pushed a commit to datastax/pulsar that referenced this pull request May 14, 2025
…es (apache#24260)

(cherry picked from commit 0350a76)
(cherry picked from commit b992578)
manas-ctds pushed a commit to datastax/pulsar that referenced this pull request May 14, 2025
…es (apache#24260)

(cherry picked from commit 0350a76)
(cherry picked from commit cfe7ac5)
manas-ctds pushed a commit to datastax/pulsar that referenced this pull request May 15, 2025
…es (apache#24260)

(cherry picked from commit 0350a76)
(cherry picked from commit cfe7ac5)
manas-ctds pushed a commit to datastax/pulsar that referenced this pull request May 15, 2025
…es (apache#24260)

(cherry picked from commit 0350a76)
(cherry picked from commit cfe7ac5)
srinath-ctds pushed a commit to datastax/pulsar that referenced this pull request May 16, 2025
…es (apache#24260)

(cherry picked from commit 0350a76)
(cherry picked from commit cfe7ac5)
srinath-ctds pushed a commit to datastax/pulsar that referenced this pull request May 18, 2025
…es (apache#24260)

(cherry picked from commit 0350a76)
(cherry picked from commit b992578)
nodece pushed a commit to ascentstream/pulsar that referenced this pull request May 28, 2025
KannarFr pushed a commit to CleverCloud/pulsar that referenced this pull request Sep 22, 2025
walkinggo pushed a commit to walkinggo/pulsar that referenced this pull request Oct 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants