Conversation
8f4c57a to
6a229e0
Compare
e4f3614 to
061c9b7
Compare
061c9b7 to
b4d66b3
Compare
|
\cc @felixcheung for review |
|
This is fixes 2 CI https://travis-ci.org/bzz/incubator-zeppelin and https://travis-ci.org/apache/incubator-zeppelin and is ready to be merged. \cc @felixcheung @Leemoonsoo @jongyoul @prabhjyotsingh for a review |
|
Merging if there is no more discussion |
|
LGTM |
| set -xe | ||
|
|
||
| TIMEOUT_SEC=590 | ||
| MAX_DOWNLOAD_TIME_SEC=590 |
There was a problem hiding this comment.
Trivial: This isn't used any longer.
There was a problem hiding this comment.
Good catch, but I thought about keeping it for a while, reminding about great use of timeout that we have discarded for now. We possibly might what get back to it one day (i.e to reduce build time), once we get the build stability.
|
LGTM |
|
Thank you for reviews! |
|
Looks good - sorry for the delay. |
### What is this PR for? Fix Spark download on CI ### What type of PR is it? Hot Fix ### What is the Jira issue? [ZEPPELIN-783](https://issues.apache.org/jira/browse/ZEPPELIN-783) ### How should this be tested? CI must be green ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Alexander Bezzubov <[email protected]> Closes apache#818 from bzz/ZEPPELIN-783-stable-ci-part-2 and squashes the following commits: b4d66b3 [Alexander Bezzubov] ZEPPELIN-783: advanced Spark download failover procedure
What is this PR for?
Fix Spark download on CI
What type of PR is it?
Hot Fix
What is the Jira issue?
ZEPPELIN-783
How should this be tested?
CI must be green
Questions: