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.

Sign up for GitHub

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

Jump to bottom

Update maven to 3.9.1 #16777

Merged
merged 1 commit into from
Mar 31, 2023
Merged

Update maven to 3.9.1 #16777

merged 1 commit into from
Mar 31, 2023

Conversation

wendigo
Copy link
Contributor

@wendigo wendigo commented Mar 29, 2023 

Apache Maven 3.9.1 (2e178502fcdbffc201671fb2537d0cb4b4cc58f8)
Maven home: /Users/mateuszgajewski/.m2/wrapper/dists/apache-maven-3.9.1-bin/6e2gfsmef58top79hk1hgrm2ot/apache-maven-3.9.1
Java version: 17.0.6, vendor: Eclipse Adoptium, runtime: /Users/mateuszgajewski/.sdkman/candidates/java/17.0.6-tem
Default locale: en_PL, platform encoding: UTF-8
OS name: "mac os x", version: "13.2.1", arch: "aarch64"

The problem with the 10% performance regression introduced in 3.9.0 is solved in 3.9.1. It should be safe to upgrade.

@cla-bot cla-bot bot added the cla-signed label Mar 29, 2023
@wendigo wendigo requested review from hashhar and kokosing March 29, 2023 09:04
@kokosing
Copy link
Member

The problem with the 10% performance regression introduced in 3.9.0 is solved in 3.9.1. It should be safe to upgrade.

Do you have any ticket or issue about it?

@wendigo wendigo mentioned this pull request Mar 29, 2023
Closed
@wendigo wendigo requested a review from martint March 29, 2023 09:08
@wendigo
Copy link
Contributor Author

wendigo commented Mar 29, 2023

@kokosing https://issues.apache.org/jira/browse/MNG-7677

@wendigo wendigo force-pushed the serafin/mvn-3.9.1 branch from 70ff416 to 3384759 Compare March 29, 2023 19:09
@wendigo wendigo force-pushed the serafin/mvn-3.9.1 branch from 3384759 to da49c2b Compare March 30, 2023 15:07
@wendigo
Copy link
Contributor Author

wendigo commented Mar 31, 2023

PTAL @kokosing @findepi

@kokosing kokosing merged commit d3a471b into trinodb:master Mar 31, 2023
@kokosing
Copy link
Member

Thank you!

@wendigo wendigo deleted the serafin/mvn-3.9.1 branch March 31, 2023 12:09
@github-actions github-actions bot added this to the 412 milestone Mar 31, 2023
@colebow colebow mentioned this pull request Apr 4, 2023
Merged
@ebyhr ebyhr mentioned this pull request Apr 26, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ksobolew ksobolew ksobolew left review comments

@kokosing kokosing kokosing approved these changes

@hashhar hashhar Awaiting requested review from hashhar

@martint martint Awaiting requested review from martint

Assignees
No one assigned
Labels
cla-signed
Milestone
412
Development

Successfully merging this pull request may close these issues.
4 participants
@wendigo @kokosing @ksobolew @hashhar