You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@celeborn.apache.org by zh...@apache.org on 2023/03/10 13:17:27 UTC

[incubator-celeborn] branch main updated: [CELEBORN-393] responseBuilder.setCmdType should be called only once in MetaHandler's handleReadRequest method (#1322)

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

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


The following commit(s) were added to refs/heads/main by this push:
     new 92ad56c47 [CELEBORN-393] responseBuilder.setCmdType should be called only once in MetaHandler's handleReadRequest method (#1322)
92ad56c47 is described below

commit 92ad56c47dee34a9dca9c2619fa983bac0d8d659
Author: jiaoqingbo <11...@qq.com>
AuthorDate: Fri Mar 10 21:17:22 2023 +0800

    [CELEBORN-393] responseBuilder.setCmdType should be called only once in MetaHandler's handleReadRequest method (#1322)
---
 .../celeborn/service/deploy/master/clustermeta/ha/MetaHandler.java      | 2 --
 1 file changed, 2 deletions(-)

diff --git a/master/src/main/java/org/apache/celeborn/service/deploy/master/clustermeta/ha/MetaHandler.java b/master/src/main/java/org/apache/celeborn/service/deploy/master/clustermeta/ha/MetaHandler.java
index 839cefa59..a890ac1ff 100644
--- a/master/src/main/java/org/apache/celeborn/service/deploy/master/clustermeta/ha/MetaHandler.java
+++ b/master/src/main/java/org/apache/celeborn/service/deploy/master/clustermeta/ha/MetaHandler.java
@@ -68,7 +68,6 @@ public class MetaHandler {
   public ResourceResponse handleReadRequest(ResourceProtos.ResourceRequest request) {
     ResourceProtos.Type cmdType = request.getCmdType();
     ResourceResponse.Builder responseBuilder = getMasterMetaResponseBuilder(request);
-    responseBuilder.setCmdType(cmdType);
     try {
       switch (cmdType) {
         default:
@@ -88,7 +87,6 @@ public class MetaHandler {
   public ResourceResponse handleWriteRequest(ResourceProtos.ResourceRequest request) {
     ResourceProtos.Type cmdType = request.getCmdType();
     ResourceResponse.Builder responseBuilder = getMasterMetaResponseBuilder(request);
-    responseBuilder.setCmdType(cmdType);
     try {
       String shuffleKey;
       String appId;