[FLINK-33543] Move Java 17-specific code into its own module to please Intellij

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.19.0
• Fix Version/s: 1.19.0
• Component/s: API / Type Serialization System, Build System
• Labels: pull-request-available
• Assignee: Matthias Pohl
• Reporter: Matthias Pohl
• Created: 14/Nov/23 12:16
• Updated: 15/Nov/23 08:35
• Resolved: 15/Nov/23 08:35

Description

FLINK-32380 introduced support for Java records (which are introduced in Java 14). Test classes were added and included via Maven profiles. This works on the command line. Intellij runs into issues though when build flink-core.

snuyanzin filed IDEA-338060 to cover this issue.

A workaround on the Flink side is to have a dedicated module for Java 17 code (more specifically the test classes which use Java record). This would make flink-core compilable again. The newly introduced module can be enabled for the java17 profile.

Issue Links

• is caused by: FLINK-32380 Support Java records
• links to: GitHub Pull Request #23709
• links to: IDEA-338060: How to make IntellijIdea aware of maven-compiler-plugin testExcludes