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

[FLINK-8113][build] Bump maven-shade-plugin to 3.0.0 #5042

Closed
wants to merge 1 commit into from

Conversation

zentol
Copy link
Contributor

@zentol zentol commented Nov 21, 2017

What is the purpose of the change

This PR bumps the maven-shade-plugin version to 3.0.0.
Benefits:

  • all modules now use the same version again
    • currently, some modules set it to 3.0.0 already to fix a problem with relocating services
  • we no longer need to bump the ASM version of the plugin

Verifying this change

This change should be covered by travis and the end-to-end tests.

@StephanEwen
Copy link
Contributor

Looks good to me.

Do we know if there are any new known issues in shade "3.0.0"? The first release of a new major version sometimes has a few regressions. Have you checked the Maven JIRA per chance?

Otherwise +1

@zentol
Copy link
Contributor Author

zentol commented Nov 21, 2017

I looked through the open issues for 3.0.0, and the ones that were fixed for 3.1.0 and i couldn't find anything critical.

@zentol
Copy link
Contributor Author

zentol commented Nov 22, 2017

merging,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants