[GitHub] [spark] deshanxiao commented on issue #27647: [SPARK-30883][TEST] Cancel the tests when setReadable and setExecutable don't work

2020-02-21 Thread GitBox
deshanxiao commented on issue #27647: [SPARK-30883][TEST] Cancel the tests when 
setReadable and setExecutable don't work
URL: https://github.com/apache/spark/pull/27647#issuecomment-589923654
 
 
   I agree. Is it necessary to add some code to prevent running as root? 
@jiangxb1987 


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

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



[GitHub] [spark] deshanxiao commented on issue #27647: [SPARK-30883][TEST] Cancel the tests when setReadable and setExecutable don't work

2020-02-20 Thread GitBox
deshanxiao commented on issue #27647: [SPARK-30883][TEST] Cancel the tests when 
setReadable and setExecutable don't work
URL: https://github.com/apache/spark/pull/27647#issuecomment-589164684
 
 
   @srowen  Sure, running tests as root is not a good idea. I run my tests in 
gitlab ci/cd. Root is its default user. Maybe, it's not bad to cancel these 
tests. Or just disable root to run all tests?


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

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