[SPARK-31587][R][INFRA] Uses R 4.0.0 in Github Actions

[HyukjinKwon]

What changes were proposed in this pull request?

Currently R installation seems being failed in Github Actions:

Get:61 https://dl.bintray.com/sbt/debian  Packages [4174 B]
Get:62 http://ppa.launchpad.net/apt-fast/stable/ubuntu bionic/main amd64 Packages [532 B]
Get:63 http://ppa.launchpad.net/git-core/ppa/ubuntu bionic/main amd64 Packages [3036 B]
Get:64 http://ppa.launchpad.net/ondrej/php/ubuntu bionic/main amd64 Packages [52.0 kB]
Get:65 http://ppa.launchpad.net/ubuntu-toolchain-r/test/ubuntu bionic/main amd64 Packages [33.9 kB]
Get:66 http://ppa.launchpad.net/ubuntu-toolchain-r/test/ubuntu bionic/main Translation-en [10.1 kB]
Reading package lists...
E: The repository 'https://cloud.r-project.org/bin/linux/ubuntu bionic-cran35/ Release' does not have a Release file.
##[error]Process completed with exit code 100.

Why are the changes needed?

To fix the broken builds in Github Actions.

Does this PR introduce any user-facing change?

No, it's dev-only change.

How was this patch tested?

Github Actions build will test it out.

[MichaelChirico]

There's another reference to this in dev/create-release/spark-rm/Dockerfile

[dongjoon-hyun]

Thank you for the investigation and quick fix, @HyukjinKwon . I also noticed the failures are consistent. Let see the GitHub Action result~

[dongjoon-hyun]

FYI, I checked the official README.

• https://cloud.r-project.org/bin/linux/ubuntu/README.html

deb https://cloud.r-project.org/bin/linux/ubuntu bionic-cran35/

[HyukjinKwon]

Yes .. I just checked too. Let me investigate a bit more.

[dongjoon-hyun]

Thank you so much. Although I have no clue yet, Ubuntu OS might be changed by GitHub Action. Both Ubuntu and R released a new version.

[HyukjinKwon]

Yup, maybe it just needs some time to sync. Thanks for pointing these out @MichaelChirico and @dongjoon-hyun.

[MichaelChirico]

While we're in this file, maybe update:

Rscript -e "devtools::install_github('jimhester/lintr')" && \

to use remotes instead? It's a more lightweight install than devtools.

Also lintr is now on CRAN:

https://cran.r-project.org/web/packages/lintr/index.html

Maybe we can drop devtools entirely and just install lintr in the line above?

[HyukjinKwon]

devtools is used in other places too, e.g. devtools::document. I would just like to stick to it rather than having another dependency specifically in the CI.

lintr was being installed from the Github directly because they didn't bump up the major version for long time, which catches a lot of more instances. We could switch to the official release instead.

The problem here is that lintr isn't portable so Jenkins has the specific version of lintr. If we want to change, we should co-op to upgrade the lintr in our Jenkins machine too.

[MichaelChirico]

OK so it's not an easy fix, should be addressed in a separate PR. NVM for now.

[HyukjinKwon]

Closing in favour of #28382. We might still have to upgrade but the main issue it targets to fix is the build.

[SparkQA]

Test build #121943 has finished for PR 28380 at commit 5dc15cd.

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

[SparkQA]

Test build #121945 has finished for PR 28380 at commit 7203521.

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