Skip to content

Commit ee37d3a

Browse files
LuciferYangdongjoon-hyun
authored andcommitted
[SPARK-51484][SS] Remove unused function private def newDFSFileName(String) from RocksDBFileManager
### What changes were proposed in this pull request? This pr aims to remove unused function `private def newDFSFileName(String)` from `RocksDBFileManager`, It is no longer used after SPARK-49770 (#47875) ### Why are the changes needed? Code cleanup. ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Pass GitHub Actions ### Was this patch authored or co-authored using generative AI tooling? No Closes #50249 from LuciferYang/SPARK-51484. Authored-by: yangjie01 <yangjie01@baidu.com> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
1 parent baf4b59 commit ee37d3a

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/RocksDBFileManager.scala

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ package org.apache.spark.sql.execution.streaming.state
2020
import java.io.{File, FileInputStream, InputStream}
2121
import java.nio.charset.StandardCharsets.UTF_8
2222
import java.nio.file.Files
23-
import java.util.UUID
2423
import java.util.concurrent.ConcurrentHashMap
2524
import java.util.zip.{ZipEntry, ZipOutputStream}
2625

@@ -870,12 +869,6 @@ class RocksDBFileManager(
870869
log"${MDC(LogKeys.FILE_NAME, files.mkString("\n\t"))}")
871870
}
872871

873-
private def newDFSFileName(localFileName: String): String = {
874-
val baseName = FilenameUtils.getBaseName(localFileName)
875-
val extension = FilenameUtils.getExtension(localFileName)
876-
s"$baseName-${UUID.randomUUID}.$extension"
877-
}
878-
879872
def newDFSFileName(localFileName: String, dfsFileSuffix: String): String = {
880873
val baseName = FilenameUtils.getBaseName(localFileName)
881874
val extension = FilenameUtils.getExtension(localFileName)

0 commit comments

Comments
 (0)