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-31589][INFRA][2.4] Use r-lib/actions/setup-r in GitHub Action #28384

Closed
wants to merge 1 commit into from
Closed

[SPARK-31589][INFRA][2.4] Use r-lib/actions/setup-r in GitHub Action #28384

wants to merge 1 commit into from

Conversation

dongjoon-hyun
Copy link
Member

@dongjoon-hyun dongjoon-hyun commented Apr 28, 2020

What changes were proposed in this pull request?

This PR aims to use r-lib/actions/setup-r because it's more stable and maintained by 3rd party.

Why are the changes needed?

This will recover the current outage. In addition, this will be more robust in the future.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass the GitHub Actions, especially Linter R and Generate Documents.

This PR aims to use `r-lib/actions/setup-r` because it's more stable and maintained by 3rd party.

This will recover the current outage. In addition, this will be more robust in the future.
As of now, this is tested via dongjoon-hyun#17 .

No.

Pass the GitHub Actions, especially `Linter R` and `Generate Documents`.

Closes #28382 from dongjoon-hyun/SPARK-31589.

Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: HyukjinKwon <[email protected]>
(cherry picked from commit 2d3e960)
Signed-off-by: HyukjinKwon <[email protected]>
@dongjoon-hyun
Copy link
Member Author

Thank you, @HyukjinKwon .

dongjoon-hyun added a commit that referenced this pull request Apr 28, 2020
### What changes were proposed in this pull request?

This PR aims to use `r-lib/actions/setup-r` because it's more stable and maintained by 3rd party.

### Why are the changes needed?

This will recover the current outage. In addition, this will be more robust in the future.

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

No.

### How was this patch tested?

Pass the GitHub Actions, especially `Linter R` and `Generate Documents`.

Closes #28384 from dongjoon-hyun/SPARK-31589-2.4.

Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
@dongjoon-hyun
Copy link
Member Author

Merged to branch-2.4 because GitHub Action passed.

@dongjoon-hyun
Copy link
Member Author

cc @holdenk since she is the release manager of 2.4.6.
(This is just for GitHub Action infra which is irrelevant to our code.)

@dongjoon-hyun dongjoon-hyun deleted the SPARK-31589-2.4 branch April 28, 2020 04:59
@SparkQA
Copy link

SparkQA commented Apr 28, 2020

Test build #121953 has finished for PR 28384 at commit 67d3423.

  • This patch fails due to an unknown error code, -9.
  • This patch merges cleanly.
  • This patch adds no public classes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants