Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19959#discussion_r156528239
--- Diff: dev/lint-r.R ---
@@ -27,10 +27,11 @@ if (! library(SparkR, lib.loc = LOCAL_LIB_LOC,
logical.return = TRUE)) {
# Installs lintr from Github in a local directory.
# NOTE: The CRAN's version is too old to adapt to our rules.
if ("lintr" %in% row.names(installed.packages()) == FALSE) {
--- End diff --
@falaki, actually, seems it's not installing `lintr` in Jenkins:
```
Error in library(lintr, lib.loc = LOCAL_LIB_LOC) :
there is no package called 'lintr'
Execution halted
```
I think it's because there's `lintr` in Jenkins already and `"lintr" %in%
row.names(installed.packages())` returns `TRUE`.
So, I think `lintr` is already not synced to `jimhester/lintr@5431140` and
I had some discussion, for example, here -
https://issues.apache.org/jira/browse/SPARK-22063
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
