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

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


The following commit(s) were added to refs/heads/main by this push:
     new 3a83ac769 [CELEBORN-1889] Fix scala 2.13 complie error
3a83ac769 is described below

commit 3a83ac769335bbd37bfbb056e7735d3788c4a2c7
Author: Chongchen Chen <[email protected]>
AuthorDate: Mon Mar 3 15:41:02 2025 +0800

    [CELEBORN-1889] Fix scala 2.13 complie error
    
    ### What changes were proposed in this pull request?
    
    add '()' to override method, because method in parent class has '()'
    
    ### Why are the changes needed?
    
    ./build/make-distribution.sh -Dscala.version=2.13.5 
-Dscala.binary.version=2.13 -Pjdk-17
    complains error
    
    ### Does this PR introduce _any_ user-facing change?
    
    No
    
    ### How was this patch tested?
    
    Unit test
    
    Closes #3127 from chenkovsky/fix/tierwriter.
    
    Authored-by: Chongchen Chen <[email protected]>
    Signed-off-by: mingji <[email protected]>
---
 .../apache/celeborn/service/deploy/worker/storage/TierWriter.scala    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/worker/src/main/scala/org/apache/celeborn/service/deploy/worker/storage/TierWriter.scala
 
b/worker/src/main/scala/org/apache/celeborn/service/deploy/worker/storage/TierWriter.scala
index a23a426e7..7112c5bb5 100644
--- 
a/worker/src/main/scala/org/apache/celeborn/service/deploy/worker/storage/TierWriter.scala
+++ 
b/worker/src/main/scala/org/apache/celeborn/service/deploy/worker/storage/TierWriter.scala
@@ -371,7 +371,7 @@ class LocalTierWriter(
   private val channel: FileChannel =
     FileChannelUtils.createWritableFileChannel(diskFileInfo.getFilePath);
 
-  override def needEvict: Boolean = {
+  override def needEvict(): Boolean = {
     false
   }
 
@@ -518,7 +518,7 @@ class DfsTierWriter(
       hadoopFs.create(hdfsFileInfo.getDfsPath, true).close()
   }
 
-  override def needEvict: Boolean = {
+  override def needEvict(): Boolean = {
     false
   }
 

Reply via email to