-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Gradle 4 upgrade #5949
Gradle 4 upgrade #5949
Conversation
@jhipster/developers I have tested all above and couldn't find any problems, so I suggest we can merge it. |
Furthermore when using gradle you can execute a single simulation with |
Congrats!! As there is a lot of advertisement in conferences and Twitter about the new Gradle performance enhancements, did you experience any speed up? |
Right that's what wanted to try with empty m2 repository as that was most annoying because dependencies have not been downloaded in parallel. Will post some numbers. |
Maybe the problem comes from this commit 635a757 and the springfox version 2.7.0 |
@pascalgrimaud You are right. It doesn't work. Get this error EDIT: Same problem on master |
yes, I saw it there : https://travis-ci.org/hipster-labs/jhipster-travis-build/branches Not perfected yet and a lot of random failures, specially with uaa, but there are more tests than our current Travis |
Wow, cool. Will check it and try it locally. |
I fixed the cassandra problem (missing guava dependency).
I have no numbers currently for gradle 3.5 but the initial build with empty caches was terrible slow because of non parallel download of dependencies, so for me these are very cool results. |
OK, so nothing extraordinary here, but we need to upgrade anyway -> are you confident with merging your PR? If yes, let's do it! |
Let's Merge it to see if any less used combination doesn't work well. As it also fixes gradle and Cassandra we should Merge it. |
Wow so finally now Gradle is faster than maven. @atomfrede what say we make Gradle the default option? |
@jdubois what say we make Gradle the default option? |
I second it. 👍 |
@deepu105 Would like to have gradle as default, but I think maven is still the dominant build system. E.g. all our services are build with maven. We use gradle for custom things, like executing test with docker or setting up our local minikube environment or handling translations via crowdin. @jdubois How are the current stats maven vs gradle? |
I'm a maven user, so I'd prefer keeping maven as default :) |
Stats will always swing towards what we set us default :)
Thanks & Regards,
Deepu
…On Mon, Jul 3, 2017 at 8:50 AM, Pascal Grimaud ***@***.***> wrote:
I'm a maven user, so I'd prefer keeping maven as default :)
I'm curious to have some stats between maven and gradle too
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#5949 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABDlFxlK-Lh6AgOeHofDSG97uIjn16-kks5sKI9IgaJpZM4N9xkM>
.
|
Yes stats are always misleading, but as you asked for them, for July 2017, we have:
Now I don't want to change the build tools for the following reasons:
Then, I'm totally opened at migrating to Gradle in the future:
|
Totally agree with you, except I don't understand the maven config very well :) But ~20% for a non default option is quite good I think. |
Gradle is designed to trade off the first startup for fast subsequent builds. The daemon is the default for that reason. Running without it only makes sense on throwaway CI machines like Travis. That being said, the only time I've seen Maven being quicker on a "cold" machine was either
Using
It's 50% faster than Maven for a clean build on a pretty simple project. That's pretty extraordinary, given that in such a small project, there is almost nothing to optimize ;) I'm not going to advocate for or against changing the default, that should be decided based on your community's feedback. I just wanted to give some perspective on those numbers. |
Thanks for the clarification @oehme |
Sure, if you point me to an example of a typical project you create, I'm happy to give some feedback. |
@oehme I think best way is to test drive https://jhipster.julien-dubois.com/. I have created a simple monolithic application here: https://github.com/atomfrede/jhipsterGradleSampleApplication If you find things to do better (I am sure you will) that would be awesome! |
Yes and there is also the official Gradle sample here https://github.com/jhipster/jhipster-sample-app-gradle which should be the same as @atomfrede posted above s it is generated with same version of JHipster |
@oehme oh yes, your feedback is very highly welcome here!!!! I can't guarantee we'll switch the default to Gradle, but if performance is much better (and maybe features also?), we'll do our best to market it, and maybe change the stats! |
Just FYI, I'm on vacation next week, so I'll only get around to this in August. I think setting up a hangout would be ideal to walk through some of the pieces. |
Yes we could do that I guess
…On 21 Jul 2017 3:03 pm, "Stefan Oehme" ***@***.***> wrote:
Just FYI, I'm on vacation next week, so I'll only get around to this in
August. I think setting up a hangout would be ideal to walk through some of
the pieces.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#5949 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABDlFxU1vbT-AE1cwoK-7QP6YpHT7Qc_ks5sQKG4gaJpZM4N9xkM>
.
|
I am also on vacation next week so august would be fine. |
Thanks a lot for the clarification @oehme |
@oehme could you let us know a date in August when you would be available for the call you suggested so that I can send out an invitation based on that so that people dont forget :) |
How about the 16th or 17th at 18:00 CEST/ 9:00 PST? From our side @eriwen and I would like to join in. |
16th is not possible for me as we have jug meetup but no problem if everyone else prefers 16th. |
@deepu105 Wednesday is now already taken for me, but the time slot on Thursday is still free. Can you send out an invite? |
@oehme yes I'll send an invite. Unfortunately, my 6PM Thursday is taken as well but if @atomfrede and @sendilkumarn are available that should be fine. Guys please confirm |
@jhipster/developers anyone else interested to join? |
I'd love to, but I'm still on holidays (with 3 small kids, so I wonder if that can be called holidays??) |
I can make it. |
@atomfrede great. I'll send an invite |
I have send out a google calendar invite for 17th August 1800 CEST |
@deepu105 There is no hangout attached to the calendar invite, can you please add one? |
@atomfrede @oehme @sendilkumarn sorry guys I didnt have access to laptop as I was traveling. Did you manage to make the Hangout? I guess I totally forgot to attach the link. damn me |
I posted a summary to our mailing list :)
2017-08-17 21:20 GMT+02:00 Deepu K Sasidharan <notifications@github.com>:
… @atomfrede <https://github.com/atomfrede> @oehme
<https://github.com/oehme> @sendilkumarn <https://github.com/sendilkumarn>
sorry guys I didnt have access to laptop as I was traveling. Did you manage
to make the Hangout? I guess I totally forgot to attach the link. damn me
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#5949 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAMaicKukxLTKiEJCFL76_-36qjxJcJxks5sZJJ2gaJpZM4N9xkM>
.
|
Pasting the thread link for anybody interested https://groups.google.com/forum/?utm_medium=email&utm_source=footer#!msg/jhipster-dev/P91z4qd_-XY/m6qQTN8jAAAJ |
I fixed some problems with dependency resolution (see comment here: spring-projects/spring-boot#9516) and tested with following settings:
I will update the PR when more tests are done,
close #5936