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

ethanfeng pushed a commit to branch branch-0.6
in repository https://gitbox.apache.org/repos/asf/celeborn.git


The following commit(s) were added to refs/heads/branch-0.6 by this push:
     new d1ef219fe [CELEBORN-2139] Fix the condition for using OSS storage
d1ef219fe is described below

commit d1ef219fe28dc02bac2ab2fc5ef9aa0053eefeb3
Author: xxx <953396...@qq.com>
AuthorDate: Wed Sep 3 15:40:42 2025 +0800

    [CELEBORN-2139] Fix the condition for using OSS storage
    
    ### What changes were proposed in this pull request?
    
    Fix the condition for using OSS storage
    
    ### Why are the changes needed?
    
    When OSS is enabled, the local disk should be used first if it is available.
    
    ### Does this PR introduce _any_ user-facing change?
    
    NO
    
    ### How was this patch tested?
    
    CI
    
    Closes #3463 from xy2953396112/CELEBORN-2139.
    
    Authored-by: xxx <953396...@qq.com>
    Signed-off-by: mingji <fengmingxiao....@alibaba-inc.com>
    (cherry picked from commit 750aeefbc6bc737257967db5772b59b4040b09cc)
    Signed-off-by: mingji <fengmingxiao....@alibaba-inc.com>
---
 .../apache/celeborn/service/deploy/worker/storage/StorageManager.scala  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/worker/src/main/scala/org/apache/celeborn/service/deploy/worker/storage/StorageManager.scala
 
b/worker/src/main/scala/org/apache/celeborn/service/deploy/worker/storage/StorageManager.scala
index 7b3f1ed8c..59a4cc1da 100644
--- 
a/worker/src/main/scala/org/apache/celeborn/service/deploy/worker/storage/StorageManager.scala
+++ 
b/worker/src/main/scala/org/apache/celeborn/service/deploy/worker/storage/StorageManager.scala
@@ -1146,7 +1146,7 @@ final private[worker] class StorageManager(conf: 
CelebornConf, workerSource: Abs
           fileName,
           s3FileInfo)
         return (s3Flusher.get, s3FileInfo, null)
-      } else if (hasOssStorage && location.getStorageInfo.OSSAvailable()) {
+      } else if (dirs.isEmpty && location.getStorageInfo.OSSAvailable()) {
         val shuffleDir =
           new Path(new Path(ossDir, conf.workerWorkingDir), 
s"$appId/$shuffleId")
         FileSystem.mkdirs(

Reply via email to