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

    https://github.com/apache/spark/pull/19141#discussion_r137425482
  
    --- Diff: 
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala 
---
    @@ -565,7 +565,6 @@ private[spark] class Client(
               distribute(jarsArchive.toURI.getPath,
                 resType = LocalResourceType.ARCHIVE,
                 destName = Some(LOCALIZED_LIB_DIR))
    -          jarsArchive.delete()
    --- End diff --
    
    You're undoing the fix for SPARK-20741. If this is causing a problem and 
you want to fix it, you need to make it so that you don't do this only when the 
specific scenario that's causing the problem happens.


---

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

Reply via email to