Skip to content

[SPARK-35233][2.4][BUILD] Switch from bintray to scala.jfrog.io for SBT download in branch 2.4 and 3.0#32352

Closed
sunchao wants to merge 1 commit intoapache:branch-2.4from
sunchao:SPARK-35233
Closed

[SPARK-35233][2.4][BUILD] Switch from bintray to scala.jfrog.io for SBT download in branch 2.4 and 3.0#32352
sunchao wants to merge 1 commit intoapache:branch-2.4from
sunchao:SPARK-35233

Conversation

@sunchao
Copy link
Member

@sunchao sunchao commented Apr 26, 2021

What changes were proposed in this pull request?

Move SBT download URL from https://dl.bintray.com/typesafe to https://scala.jfrog.io/artifactory.

Why are the changes needed?

As bintray is sunsetting, we should migrate SBT download location away from it.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Tested manually and it is working, while previously it was failing because SBT 0.13.17 can't be found:

Attempting to fetch sbt
Our attempt to download sbt locally to build/sbt-launch-0.13.17.jar failed. Please install sbt manually from http://www.scala-sbt.org/

@sunchao
Copy link
Member Author

sunchao commented Apr 26, 2021

cc @viirya

@SparkQA
Copy link

SparkQA commented Apr 26, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42488/

@SparkQA
Copy link

SparkQA commented Apr 26, 2021

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42488/

Copy link
Member

@viirya viirya left a comment

Choose a reason for hiding this comment

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

LGTM. Verified locally with the change.

@SparkQA
Copy link

SparkQA commented Apr 26, 2021

Test build #137968 has finished for PR 32352 at commit 930611b.

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

@AmplabJenkins
Copy link

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/137968/

@viirya
Copy link
Member

viirya commented Apr 26, 2021

Thanks @sunchao! Merging to 2.4.

viirya pushed a commit that referenced this pull request Apr 26, 2021
…BT download in branch 2.4 and 3.0

### What changes were proposed in this pull request?

Move SBT download URL from `https://dl.bintray.com/typesafe` to `https://scala.jfrog.io/artifactory`.

### Why are the changes needed?

As [bintray is sunsetting](https://jfrog.com/blog/into-the-sunset-bintray-jcenter-gocenter-and-chartcenter/), we should migrate SBT download location away from it.

### Does this PR introduce _any_ user-facing change?

No

### How was this patch tested?

Tested manually and it is working, while previously it was failing because SBT 0.13.17 can't be found:
```
Attempting to fetch sbt
Our attempt to download sbt locally to build/sbt-launch-0.13.17.jar failed. Please install sbt manually from http://www.scala-sbt.org/
```

Closes #32352 from sunchao/SPARK-35233.

Authored-by: Chao Sun <sunchao@apple.com>
Signed-off-by: Liang-Chi Hsieh <viirya@gmail.com>
@viirya viirya closed this Apr 26, 2021
@dongjoon-hyun
Copy link
Member

+1, LGTM.

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.

5 participants