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

Add maven tools #3387

Closed
wants to merge 1 commit into from
Closed

Add maven tools #3387

wants to merge 1 commit into from

Conversation

CalvinKirs
Copy link
Member

Descriptions of the changes in this PR:

Using maven-wrapper can save users from installing maven,
and can lock the version, at the same time, improve our build time. 3.X's maven-warpper performance has been greatly improved.

Using `maven-wrapper` can save users from installing maven,
and can lock the version, at the same time, improve our build time. 3.X's `maven-warpper` performance has been greatly improved.
Copy link
Member

@StevenLuMT StevenLuMT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

improve build time from *** to ***
do you have some test data?

@hangc0276 hangc0276 added this to the 4.16.0 milestone Jul 25, 2022
@CalvinKirs
Copy link
Member Author

improve build time from *** to *** do you have some test data?

He has about 10% improvement on my M1 machine, which is also an optimization made by the official.

@hangc0276 hangc0276 modified the milestones: 4.16.0, 4.17.0 Jul 29, 2022
@StevenLuMT
Copy link
Member

fix old workflow,please see #3455 for detail

Copy link
Contributor

@nicoloboschi nicoloboschi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@CalvinKirs could you rebase ?

@hangc0276
Copy link
Contributor

@CalvinKirs Thanks for your contribution. Would you please rebase the master and resolve the conflicts? Thanks a lot. After those conflicts are resolved, I will move back to the 4.16.0.

@shoothzj
Copy link
Member

@CalvinKirs Are you still want to working on this? Could you please rebase the master code? Thanks

@shoothzj
Copy link
Member

closed by no updte, feel free to reopen it if you still want to working on this.

@shoothzj shoothzj closed this Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants