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

rickyma pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/uniffle.git


The following commit(s) were added to refs/heads/master by this push:
     new 475022b33 [#1651] fix(netty): Set Netty as the default server type 
(#2421)
475022b33 is described below

commit 475022b33245478d6cd73f78e8f981f53c258df0
Author: zhengchenyu <[email protected]>
AuthorDate: Tue Apr 1 11:19:05 2025 +0800

    [#1651] fix(netty): Set Netty as the default server type (#2421)
    
    ### What changes were proposed in this pull request?
    
    When Netty is closed, the Netty port is not passed
    
    ### Why are the changes needed?
    
    Fix mr integration test
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    integration test.
---
 server/src/main/java/org/apache/uniffle/server/ShuffleServer.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/server/src/main/java/org/apache/uniffle/server/ShuffleServer.java 
b/server/src/main/java/org/apache/uniffle/server/ShuffleServer.java
index ac2481aad..35b4ebb4c 100644
--- a/server/src/main/java/org/apache/uniffle/server/ShuffleServer.java
+++ b/server/src/main/java/org/apache/uniffle/server/ShuffleServer.java
@@ -575,7 +575,7 @@ public class ShuffleServer {
   }
 
   public int getNettyPort() {
-    return nettyPort;
+    return nettyServerEnabled ? nettyPort : -1;
   }
 
   public int getJettyPort() {

Reply via email to