Github user HyukjinKwon commented on a diff in the pull request:

    https://github.com/apache/spark/pull/23055#discussion_r237715420
  
    --- Diff: python/pyspark/worker.py ---
    @@ -22,7 +22,12 @@
     import os
     import sys
     import time
    -import resource
    +# 'resource' is a Unix specific module.
    +has_resource_module = True
    +try:
    +    import resource
    +except ImportError:
    +    has_resource_module = False
    --- End diff --
    
    What I meant by "What do you mean "python can't fail"? is quoted from 
`Python should not fail if resource cannot be imported` from @rdblue.
    
    I left the last comment before seeing your comment @vanzin. Let me leave an 
answer for that soon.



---

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

Reply via email to