Skip to content

Recognize newer and future JDK versions#350

Merged
slawekjaranowski merged 1 commit intomasterfrom
jdk-version-improvements
Mar 13, 2026
Merged

Recognize newer and future JDK versions#350
slawekjaranowski merged 1 commit intomasterfrom
jdk-version-improvements

Conversation

@slawekjaranowski
Copy link
Copy Markdown
Member

No description provided.

@slawekjaranowski
Copy link
Copy Markdown
Member Author

@wendigo please look

@@ -114,33 +114,6 @@ public class EnforceBytecodeVersion extends AbstractResolveDependencies {
// Java 17
JDK_TO_MAJOR_VERSION_NUMBER_MAPPING.put("17", 61);
JDK_TO_MAJOR_VERSION_NUMBER_MAPPING.put("1.17", 61);
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I'm not sure about such mappings, like 1.17

@slawekjaranowski slawekjaranowski merged commit d5f20f1 into master Mar 13, 2026
14 checks passed
@slawekjaranowski slawekjaranowski deleted the jdk-version-improvements branch March 13, 2026 16:43
sarutak pushed a commit to apache/spark that referenced this pull request Apr 21, 2026
### What changes were proposed in this pull request?

This PR aims to upgrade `extra-enforcer-rules` to 1.12.0.

### Why are the changes needed?

To bring the latest bug fixes and improvements. This plugin dependency powers the `EnforceBytecodeVersion` rule used by our `enforce-versions` enforcer execution, which guards against binaries built for an incompatible JDK target.

- https://github.com/mojohaus/extra-enforcer-rules/releases/tag/1.12.0
  - mojohaus/extra-enforcer-rules#350

### Does this PR introduce _any_ user-facing change?

No. This is a build-only dependency change.

### How was this patch tested?

Pass the CIs.

### Was this patch authored or co-authored using generative AI tooling?

Generated-by: Claude Code (claude-opus-4-7)

Closes #55431 from dongjoon-hyun/SPARK-56553.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Kousuke Saruta <sarutak@apache.org>
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.

2 participants