(spark) branch master updated: [SPARK-53206][CORE] Use `SparkFileUtils.move` instead of `com.google.common.io.Files.move`

[dongjoon]

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

dongjoon 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 babc78d1ffbf [SPARK-53206][CORE] Use `SparkFileUtils.move` instead of 
`com.google.common.io.Files.move`
babc78d1ffbf is described below

commit babc78d1ffbf150d5be7672bea75752314902a1e
Author: Dongjoon Hyun <dongj...@apache.org>
AuthorDate: Fri Aug 8 11:18:18 2025 -0700

    [SPARK-53206][CORE] Use `SparkFileUtils.move` instead of 
`com.google.common.io.Files.move`
    
    ### What changes were proposed in this pull request?
    
    This PR aims to use `SparkFileUtils.move` instead of 
`com.google.common.io.Files.move`.
    
    ### Why are the changes needed?
    
    For consistency.
    
    ### Does this PR introduce _any_ user-facing change?
    
    No behavior change.
    
    ### How was this patch tested?
    
    Pass the CIs.
    
    ### Was this patch authored or co-authored using generative AI tooling?
    
    No.
    
    Closes #51933 from dongjoon-hyun/SPARK-53206.
    
    Authored-by: Dongjoon Hyun <dongj...@apache.org>
    Signed-off-by: Dongjoon Hyun <dongj...@apache.org>
---
 .../scala/org/apache/spark/util/logging/RollingFileAppender.scala     | 4 +---
 .../scala/org/apache/spark/executor/ExecutorClassLoaderSuite.scala    | 2 +-
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git 
a/core/src/main/scala/org/apache/spark/util/logging/RollingFileAppender.scala 
b/core/src/main/scala/org/apache/spark/util/logging/RollingFileAppender.scala
index 4967211ba97d..544ab1711fa3 100644
--- 
a/core/src/main/scala/org/apache/spark/util/logging/RollingFileAppender.scala
+++ 
b/core/src/main/scala/org/apache/spark/util/logging/RollingFileAppender.scala
@@ -20,8 +20,6 @@ package org.apache.spark.util.logging
 import java.io._
 import java.util.zip.GZIPOutputStream
 
-import com.google.common.io.Files
-
 import org.apache.spark.SparkConf
 import org.apache.spark.internal.LogKeys._
 import org.apache.spark.internal.config
@@ -100,7 +98,7 @@ private[spark] class RollingFileAppender(
         Utils.closeQuietly(gzOutputStream)
       }
     } else {
-      Files.move(activeFile, rolloverFile)
+      Utils.moveFile(activeFile, rolloverFile)
     }
   }
 
diff --git 
a/core/src/test/scala/org/apache/spark/executor/ExecutorClassLoaderSuite.scala 
b/core/src/test/scala/org/apache/spark/executor/ExecutorClassLoaderSuite.scala
index 1573d0286a32..22a3ab075cf8 100644
--- 
a/core/src/test/scala/org/apache/spark/executor/ExecutorClassLoaderSuite.scala
+++ 
b/core/src/test/scala/org/apache/spark/executor/ExecutorClassLoaderSuite.scala
@@ -105,7 +105,7 @@ class ExecutorClassLoaderSuite
     assert(result.exists(), "Compiled file not found: " + 
result.getAbsolutePath)
 
     val out = new File(scalaDir, filename)
-    Files.move(result, out)
+    Utils.moveFile(result, out)
     assert(out.exists(), "Destination file not moved: " + out.getAbsolutePath)
 
     // construct class loader tree


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