This is an automated email from the ASF dual-hosted git repository.

gurwls223 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new 4e01ebe60cc [SPARK-37677][CORE] Unzip could keep file permissions
4e01ebe60cc is described below

commit 4e01ebe60cc1a909a3492c10262231c6a086b867
Author: jokercurry <982458...@qq.com>
AuthorDate: Tue Mar 28 14:26:39 2023 +0900

    [SPARK-37677][CORE] Unzip could keep file permissions
    
    ### What changes were proposed in this pull request?
    
    Just remove comment.
    
    ### Why are the changes needed?
    
    After https://github.com/apache/hadoop/pull/4036, unzip could unzip could 
keep file permissions.
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    No need, has been added in hadoop-client side.
    
    Closes #40572 from smallzhongfeng/SPARK-37677.
    
    Authored-by: jokercurry <982458...@qq.com>
    Signed-off-by: Hyukjin Kwon <gurwls...@apache.org>
---
 core/src/main/scala/org/apache/spark/util/Utils.scala | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/core/src/main/scala/org/apache/spark/util/Utils.scala 
b/core/src/main/scala/org/apache/spark/util/Utils.scala
index d81e85ffe08..a6a1f7d7b1e 100644
--- a/core/src/main/scala/org/apache/spark/util/Utils.scala
+++ b/core/src/main/scala/org/apache/spark/util/Utils.scala
@@ -589,7 +589,7 @@ private[spark] object Utils extends Logging {
     if (lowerSrc.endsWith(".jar")) {
       RunJar.unJar(source, dest, RunJar.MATCH_ANY)
     } else if (lowerSrc.endsWith(".zip")) {
-      // TODO(SPARK-37677): should keep file permissions. Java implementation 
doesn't.
+      // After issue HADOOP-18145, unzip could keep file permissions.
       FileUtil.unZip(source, dest)
     } else if (lowerSrc.endsWith(".tar.gz") || lowerSrc.endsWith(".tgz")) {
       FileUtil.unTar(source, dest)


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

Reply via email to