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

chengpan pushed a commit to branch branch-0.3
in repository https://gitbox.apache.org/repos/asf/incubator-celeborn.git


The following commit(s) were added to refs/heads/branch-0.3 by this push:
     new 3131afa34 [CELEBORN-689] Fix the incorrect part of PushDataHandler 
message type converted to status code
3131afa34 is described below

commit 3131afa34abec772af33ad84fe773b5af72ee7d0
Author: sychen <[email protected]>
AuthorDate: Mon Jun 19 11:25:08 2023 +0800

    [CELEBORN-689] Fix the incorrect part of PushDataHandler message type 
converted to status code
    
    ### What changes were proposed in this pull request?
    
    ### Why are the changes needed?
    
    ### Does this PR introduce _any_ user-facing change?
    
    ### How was this patch tested?
    
    Closes #1601 from cxzl25/CELEBORN-689.
    
    Authored-by: sychen <[email protected]>
    Signed-off-by: Cheng Pan <[email protected]>
    (cherry picked from commit 4cb4701edef28cabe2d58369e10727eeeb5cd4a8)
    Signed-off-by: Cheng Pan <[email protected]>
---
 .../org/apache/celeborn/service/deploy/worker/PushDataHandler.scala     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/worker/src/main/scala/org/apache/celeborn/service/deploy/worker/PushDataHandler.scala
 
b/worker/src/main/scala/org/apache/celeborn/service/deploy/worker/PushDataHandler.scala
index 8b21e7d82..91ddc16f3 100644
--- 
a/worker/src/main/scala/org/apache/celeborn/service/deploy/worker/PushDataHandler.scala
+++ 
b/worker/src/main/scala/org/apache/celeborn/service/deploy/worker/PushDataHandler.scala
@@ -991,7 +991,7 @@ class PushDataHandler extends BaseMessageHandler with 
Logging {
 
     val (messageMaster, messageSlave) =
       messageType match {
-        case Type.PUSH_DATA | Type.PUSH_DATA_HAND_SHAKE =>
+        case Type.PUSH_DATA =>
           (
             StatusCode.PUSH_DATA_WRITE_FAIL_MASTER,
             StatusCode.PUSH_DATA_WRITE_FAIL_SLAVE)

Reply via email to