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

[SPARK-9633] [BUILD] SBT download locations outdated; need an update #7956

Closed
wants to merge 1 commit into from

Conversation

srowen
Copy link
Member

@srowen srowen commented Aug 5, 2015

Remove 2 defunct SBT download URLs and replace with the 1 known download URL. Also, use https.
Follow up on #7792

@SparkQA
Copy link

SparkQA commented Aug 5, 2015

Test build #39848 has finished for PR 7956 at commit caa40bd.

  • This patch fails to build.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Aug 5, 2015

Test build #1360 has finished for PR 7956 at commit caa40bd.

  • This patch fails to build.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Aug 5, 2015

Test build #1361 has finished for PR 7956 at commit caa40bd.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@liancheng
Copy link
Contributor

retest this please

@liancheng
Copy link
Contributor

LGTM.

@SparkQA
Copy link

SparkQA commented Aug 5, 2015

Test build #1362 has finished for PR 7956 at commit caa40bd.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Aug 5, 2015

Test build #39882 timed out for PR 7956 at commit caa40bd after a configured wait of 175m.

@SparkQA
Copy link

SparkQA commented Aug 5, 2015

Test build #1368 has finished for PR 7956 at commit caa40bd.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Aug 5, 2015

Test build #1370 has finished for PR 7956 at commit caa40bd.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@srowen
Copy link
Member Author

srowen commented Aug 6, 2015

I'd like to merge this even though it's a build change, on the grounds that it probably prevents flakiness -- right now the SBT download process will always need to wait to fail the first download and follow a redirect on the second to get the jar. Any failure mode that results would be quite obviously related to the download. And we should get in the HTTPS change soon, for 1.5.

asfgit pushed a commit that referenced this pull request Aug 6, 2015
Remove 2 defunct SBT download URLs and replace with the 1 known download URL. Also, use https.
Follow up on #7792

Author: Sean Owen <sowen@cloudera.com>

Closes #7956 from srowen/SPARK-9633 and squashes the following commits:

caa40bd [Sean Owen] Remove 2 defunct SBT download URLs and replace with the 1 known download URL. Also, use https.

(cherry picked from commit 681e302)
Signed-off-by: Sean Owen <sowen@cloudera.com>
asfgit pushed a commit that referenced this pull request Aug 6, 2015
Remove 2 defunct SBT download URLs and replace with the 1 known download URL. Also, use https.
Follow up on #7792

Author: Sean Owen <sowen@cloudera.com>

Closes #7956 from srowen/SPARK-9633 and squashes the following commits:

caa40bd [Sean Owen] Remove 2 defunct SBT download URLs and replace with the 1 known download URL. Also, use https.

(cherry picked from commit 681e302)
Signed-off-by: Sean Owen <sowen@cloudera.com>
asfgit pushed a commit that referenced this pull request Aug 6, 2015
Remove 2 defunct SBT download URLs and replace with the 1 known download URL. Also, use https.
Follow up on #7792

Author: Sean Owen <sowen@cloudera.com>

Closes #7956 from srowen/SPARK-9633 and squashes the following commits:

caa40bd [Sean Owen] Remove 2 defunct SBT download URLs and replace with the 1 known download URL. Also, use https.

(cherry picked from commit 681e302)
Signed-off-by: Sean Owen <sowen@cloudera.com>
@asfgit asfgit closed this in 681e302 Aug 6, 2015
@srowen srowen deleted the SPARK-9633 branch August 7, 2015 08:04
asfgit pushed a commit that referenced this pull request Aug 10, 2015
Remove 2 defunct SBT download URLs and replace with the 1 known download URL. Also, use https.
Follow up on #7792

Author: Sean Owen <sowen@cloudera.com>

Closes #7956 from srowen/SPARK-9633 and squashes the following commits:

caa40bd [Sean Owen] Remove 2 defunct SBT download URLs and replace with the 1 known download URL. Also, use https.

Conflicts:
	sbt/sbt-launch-lib.bash
asfgit pushed a commit that referenced this pull request Aug 10, 2015
Remove 2 defunct SBT download URLs and replace with the 1 known download URL. Also, use https.
Follow up on #7792

Author: Sean Owen <sowen@cloudera.com>

Closes #7956 from srowen/SPARK-9633 and squashes the following commits:

caa40bd [Sean Owen] Remove 2 defunct SBT download URLs and replace with the 1 known download URL. Also, use https.

Conflicts:
	sbt/sbt-launch-lib.bash
asfgit pushed a commit that referenced this pull request Aug 10, 2015
Remove 2 defunct SBT download URLs and replace with the 1 known download URL. Also, use https.
Follow up on #7792

Author: Sean Owen <sowen@cloudera.com>

Closes #7956 from srowen/SPARK-9633 and squashes the following commits:

caa40bd [Sean Owen] Remove 2 defunct SBT download URLs and replace with the 1 known download URL. Also, use https.

Conflicts:
	sbt/sbt-launch-lib.bash
kayousterhout pushed a commit to kayousterhout/spark-1 that referenced this pull request Aug 27, 2015
Remove 2 defunct SBT download URLs and replace with the 1 known download URL. Also, use https.
Follow up on apache#7792

Author: Sean Owen <sowen@cloudera.com>

Closes apache#7956 from srowen/SPARK-9633 and squashes the following commits:

caa40bd [Sean Owen] Remove 2 defunct SBT download URLs and replace with the 1 known download URL. Also, use https.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants