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

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


The following commit(s) were added to refs/heads/master by this push:
     new 5a05f672f710 [SPARK-52792][CORE][FOLLOWUP] Remove redundant condition 
check
5a05f672f710 is described below

commit 5a05f672f71076aa507b9ddfff46a8d3ab2328c1
Author: Dongjoon Hyun <dongj...@apache.org>
AuthorDate: Wed Jul 16 18:07:35 2025 -0700

    [SPARK-52792][CORE][FOLLOWUP] Remove redundant condition check
    
    ### What changes were proposed in this pull request?
    
    This PR is a follow-up to clean up a redundant condition check.
    
    ### Why are the changes needed?
    
    To remove a redundant condition
    - https://github.com/apache/spark/pull/51476/files#r2209040510
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    Pass the CIs.
    
    ### Was this patch authored or co-authored using generative AI tooling?
    
    No.
    
    Closes #51524 from dongjoon-hyun/SPARK-52792.
    
    Authored-by: Dongjoon Hyun <dongj...@apache.org>
    Signed-off-by: Dongjoon Hyun <dongj...@apache.org>
---
 .../src/main/java/org/apache/spark/network/server/TransportServer.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/common/network-common/src/main/java/org/apache/spark/network/server/TransportServer.java
 
b/common/network-common/src/main/java/org/apache/spark/network/server/TransportServer.java
index 19e591d00e6a..842dc7058a48 100644
--- 
a/common/network-common/src/main/java/org/apache/spark/network/server/TransportServer.java
+++ 
b/common/network-common/src/main/java/org/apache/spark/network/server/TransportServer.java
@@ -105,7 +105,7 @@ public class TransportServer implements Closeable {
       conf.getModuleName() + "-server");
 
     String name = System.getProperty("os.name");
-    boolean isNotWindows = 7 > name.length() || !name.regionMatches(true, 0, 
"Windows", 0, 7);
+    boolean isNotWindows = !name.regionMatches(true, 0, "Windows", 0, 7);
     bootstrap = new ServerBootstrap()
       .group(bossGroup, workerGroup)
       .channel(NettyUtils.getServerChannelClass(ioMode))


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to