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
[FLINK-11144][tests] Make Tests runnable on Java 9 #7293
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 with one change.
pom.xml
Outdated
@@ -1602,7 +1616,7 @@ under the License. | |||
<plugin> | |||
<groupId>org.apache.maven.plugins</groupId> | |||
<artifactId>maven-shade-plugin</artifactId> | |||
<version>3.0.0</version> | |||
<version>3.1.1</version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please move this into the jdk9 profile; I don't have the time right now to verify that the shading still works correctly for all modules.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
Note that I would propose changing the commit and JIRA title as the current one implies that we've already dealt with all issues preventing tests to complete successfully. |
Thanks for reviewing, @zentol. Merging. |
This enables to run tests on Java 9. However, not all tests are passing at the moment. This closes apache#7293.
With this change, I failed the building. |
@jinxing64 If you haven't already done so, please create an issue in jira with more details. |
What is the purpose of the change
Make tests run on Java 9 by upgrading plugins to minimum supported version for Java 9: https://cwiki.apache.org/confluence/display/MAVEN/Java+9+-+Jigsaw
cc: @zentol
Brief change log
Verifying this change
This change is already covered by existing tests, such as all tests.
Does this pull request potentially affect one of the following parts:
@Public(Evolving)
: (yes / no)Documentation