Adding travis_wait to mvn install #281
Conversation
Why ? Building such big APIs as Adyen APIs can take a lot of time. It's possible to get a time out. Here (4hwc/4HWCAutonomousCar@96c6396), I got a time out due to generating javadoc. Then, I decided to add travis_wait and it works pretty well. Official docs : https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received
FanJups
requested review from
AlexandrosMor,
martinsrenato,
rikterbeek and
rkewlani
as code owners
|
Thanks @FanJups for the PR and suggestion. |
|
You're welcome @rkewlani , my PR is based on what I faced while generating Javadoc weeks ago. I thought it would be helpful to prevent such case. |
|
Hi @FanJups, Thanks for this in general it is indeed a good idea to do this with calls that take time to produce the output. @rkewlani more info here https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received We will discuss in the team if we see any negatives if not we will merge it. Regards, |
|
Hi @FanJups, Discussed in the team and the build should not take more then 10 minutes and if it does we want to optimize this to make it shorter as it is a lightweight repo. We are building the project always for releases and did not have any issues. Could you investigate on your end what is going wrong with the generating of Javadoc as this should be quicker. Regards, |
Description
Why ?
Building such big APIs as Adyen APIs can take a lot of time. It's possible to get a time out.
Here , I got a time out due to generating javadoc.
Then, I decided to add travis_wait and it works pretty well.
Official docs : https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received