Github user HyukjinKwon commented on the issue:

    https://github.com/apache/spark/pull/20339
  
    This was tested:
    
    1. 
https://github.com/apache/spark/pull/20339/commits/fe59c8aa2b6e66930ef1fade198d35eeb27210ef
 intentionally added some changes in `.lintr` and `lint-r` to see if triggers 
the lint:   
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86430/console
    
    2. Added the detection logic to check if it triggers the lintr: 
https://github.com/apache/spark/pull/20339/commits/12789c759be18d81335600a1dd54eeb33ce76cf2
 - 
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86431/console
    
    
    
    3. Reverted the change of 1. back to leave only 2. change in this PR.
    
    cc @felixcheung and @JoshRosen, could you take a look please?


---

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

Reply via email to