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 dd817b267 [CELEBORN-956] Modify parameter passing in 
AbstractRemoteShuffleInputGateFactory
dd817b267 is described below

commit dd817b267e6eb562defc7ffefc6c58a2c3ff291c
Author: jiaoqingbo <[email protected]>
AuthorDate: Fri Sep 8 18:09:48 2023 +0800

    [CELEBORN-956] Modify parameter passing in 
AbstractRemoteShuffleInputGateFactory
    
    ### What changes were proposed in this pull request?
    
    As Title
    
    ### Why are the changes needed?
    
    As Title
    
    ### Does this PR introduce _any_ user-facing change?
    
    NO
    
    ### How was this patch tested?
    
    PASS GA
    
    Closes #1887 from jiaoqingbo/956.
    
    Authored-by: jiaoqingbo <[email protected]>
    Signed-off-by: zky.zhoukeyong <[email protected]>
---
 .../celeborn/plugin/flink/AbstractRemoteShuffleInputGateFactory.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/client-flink/common/src/main/java/org/apache/celeborn/plugin/flink/AbstractRemoteShuffleInputGateFactory.java
 
b/client-flink/common/src/main/java/org/apache/celeborn/plugin/flink/AbstractRemoteShuffleInputGateFactory.java
index c08d4e4de..c3bc6c298 100644
--- 
a/client-flink/common/src/main/java/org/apache/celeborn/plugin/flink/AbstractRemoteShuffleInputGateFactory.java
+++ 
b/client-flink/common/src/main/java/org/apache/celeborn/plugin/flink/AbstractRemoteShuffleInputGateFactory.java
@@ -73,7 +73,7 @@ public abstract class AbstractRemoteShuffleInputGateFactory {
           String.format(
               "Insufficient network memory per input gate, please increase %s 
to at " + "least %s.",
               CelebornConf.CLIENT_MEMORY_PER_INPUT_GATE().key(),
-              celebornConf.clientFlinkMemoryPerInputGate()));
+              celebornConf.clientFlinkMemoryPerInputGateMin()));
     }
 
     this.numBuffersPerGate = Utils.checkedDownCast(configuredMemorySize / 
networkBufferSize);

Reply via email to