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

Upgrade maven to 3.9.0 #16186

Closed
wants to merge 2 commits into from
Closed

Conversation

wendigo
Copy link
Contributor

@wendigo wendigo commented Feb 20, 2023

On top of the #12661

Maven 3.9.0 release notes: https://maven.apache.org/docs/3.9.0/release-notes.html

@cla-bot cla-bot bot added the cla-signed label Feb 20, 2023
@martint
Copy link
Member

martint commented Feb 20, 2023

There seems to be a known performance issue with Maven 3.9.0. We should make sure Trino is not affected: https://issues.apache.org/jira/browse/MNG-7677

@wendigo
Copy link
Contributor Author

wendigo commented Feb 20, 2023

@martint I'm gonna check those numbers for us

@wendigo
Copy link
Contributor Author

wendigo commented Feb 21, 2023

Before: Total time: 04:08 min (Wall Clock)
After: Total time: Total time: 04:04 min (Wall Clock)

It's comparable I'd say.

@wendigo
Copy link
Contributor Author

wendigo commented Mar 29, 2023

Replaced by #16777

@wendigo wendigo closed this Mar 29, 2023
@wendigo wendigo deleted the serafin/mvn-3.9.0 branch March 29, 2023 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants