Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Build] Specify release in maven-compiler-plugin configuration on JDK11 #10343

Merged
merged 2 commits into from
Apr 25, 2021

Conversation

lhotari
Copy link
Member

@lhotari lhotari commented Apr 23, 2021

Fixes #8445

Motivation

Specifying "release" parameter for maven-compiler-plugin is required on JDK11 to fix binary compatibility with JDK 8.
This blog post describes the problem and the solution: https://www.morling.dev/blog/bytebuffer-and-the-dreaded-nosuchmethoderror/

Modifications

  • rename jdk11-tests profile to jdk11
  • add maven.compiler.release property that matches the maven.compiler.target property value
  • add a workaround for configuring the maven-compiler-plugin since setting maven.compiler.release property alone didn't configure the plugin properly

Copy link
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@lhotari
Copy link
Member Author

lhotari commented Apr 23, 2021

/pulsarbot run-failure-checks

@merlimat merlimat added this to the 2.8.0 milestone Apr 23, 2021
@lhotari
Copy link
Member Author

lhotari commented Apr 23, 2021

/pulsarbot run-failure-checks

3 similar comments
@lhotari
Copy link
Member Author

lhotari commented Apr 24, 2021

/pulsarbot run-failure-checks

@lhotari
Copy link
Member Author

lhotari commented Apr 24, 2021

/pulsarbot run-failure-checks

@lhotari
Copy link
Member Author

lhotari commented Apr 24, 2021

/pulsarbot run-failure-checks

@merlimat merlimat merged commit e5a2c5f into apache:master Apr 25, 2021
eolivelli pushed a commit to datastax/pulsar that referenced this pull request May 14, 2021
…11 (apache#10343)

* Specify release in maven-compiler-plugin configuration

* Use "8" instead of "1.8"

(cherry picked from commit e5a2c5f)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants