Skip to content

Commit 91d9901

Browse files
committed
[SPARK-24152][R][TESTS] Disable check-cran from run-tests.sh
### What changes were proposed in this pull request? This PR aims to remove `check-cran` from `run-tests.sh`. We had better add an independent Jenkins job to run `check-cran`. ### Why are the changes needed? CRAN instability has been a blocker for our daily dev process. The following simple check causes consecutive failures in 4 of 9 Jenkins jobs + PR builder. ``` * checking CRAN incoming feasibility ...Error in .check_package_CRAN_incoming(pkgdir) : dims [product 24] do not match the length of object [0] ``` - spark-branch-2.4-test-sbt-hadoop-2.6 - spark-branch-2.4-test-sbt-hadoop-2.7 - spark-master-test-sbt-hadoop-2.7 - spark-master-test-sbt-hadoop-3.2 - PRBuilder ### Does this PR introduce any user-facing change? No. ### How was this patch tested? Currently, PR builder is failing due to the above issue. This PR should pass the Jenkins. Closes apache#26375 from dongjoon-hyun/SPARK-24152. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
1 parent be022d9 commit 91d9901

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

R/run-tests.sh

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,10 @@ NUM_TEST_WARNING="$(grep -c -e 'Warnings ----------------' $LOGFILE)"
3232
CRAN_CHECK_LOG_FILE=$FWDIR/cran-check.out
3333
rm -f $CRAN_CHECK_LOG_FILE
3434

35-
NO_TESTS=1 NO_MANUAL=1 $FWDIR/check-cran.sh 2>&1 | tee -a $CRAN_CHECK_LOG_FILE
36-
FAILED=$((PIPESTATUS[0]||$FAILED))
35+
# SPARK-24152 We will add this test back as a separate Jenkins job
36+
# NO_TESTS=1 NO_MANUAL=1 $FWDIR/check-cran.sh 2>&1 | tee -a $CRAN_CHECK_LOG_FILE
37+
# FAILED=$((PIPESTATUS[0]||$FAILED))
38+
touch $CRAN_CHECK_LOG_FILE
3739

3840
NUM_CRAN_WARNING="$(grep -c WARNING$ $CRAN_CHECK_LOG_FILE)"
3941
NUM_CRAN_ERROR="$(grep -c ERROR$ $CRAN_CHECK_LOG_FILE)"

0 commit comments

Comments
 (0)