You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by shaneknapp <gi...@git.apache.org> on 2018/10/30 18:03:18 UTC

[GitHub] spark issue #22896: [SPARKR]found some extra whitespace in the R tests

Github user shaneknapp commented on the issue:

    https://github.com/apache/spark/pull/22896
  
    i am confused, though, as to why this wasn't caught during the PR that added this file:
    https://github.com/apache/spark/pull/22455
    
    on the centos workers, there was an error message in the PRB builds for that PR, but they didn't stop execution:
    Error: StartTag: invalid element name [68]
    
    however, it *was* caught on the ubuntu workers.
    
    both centos and ubuntu are running the same version of lintr and testthat.
    
    centos:
    ```
    [sknapp@amp-jenkins-worker-04 ~]$ Rscript -e "packageVersion('lintr')"
    [1] ‘1.0.0.9001’
    [sknapp@amp-jenkins-worker-04 ~]$ Rscript -e "packageVersion('testthat')"
    [1] ‘1.0.2’
    ```
    
    ubuntu:
    ```
    sknapp@amp-jenkins-staging-worker-02:~$ Rscript -e "packageVersion('lintr')"
    [1] ‘1.0.0.9001’
    sknapp@amp-jenkins-staging-worker-02:~$ Rscript -e "packageVersion('testthat')"
    [1] ‘1.0.2’
    ```
    
    i'll investigate further, but....    ¯\_(ツ)_/¯


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org