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

zuston 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 c7e23b6b7 chore: Fix grammar in RssException message (#2695)
c7e23b6b7 is described below

commit c7e23b6b72b573f1a5019a41c1ca16a36cc3932a
Author: Mark Wadham <[email protected]>
AuthorDate: Wed Dec 10 02:48:30 2025 +0000

    chore: Fix grammar in RssException message (#2695)
    
    ### What changes were proposed in this pull request?
    
    Correction of grammar.
    
    ### Why are the changes needed?
    
    So that the grammar will be correct.
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    N/A
---
 .../strategy/assignment/PartitionBalanceAssignmentStrategy.java         | 2 +-
 .../src/test/java/org/apache/uniffle/test/RssShuffleManagerTest.java    | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/coordinator/src/main/java/org/apache/uniffle/coordinator/strategy/assignment/PartitionBalanceAssignmentStrategy.java
 
b/coordinator/src/main/java/org/apache/uniffle/coordinator/strategy/assignment/PartitionBalanceAssignmentStrategy.java
index 402df7799..5fb6b87f0 100644
--- 
a/coordinator/src/main/java/org/apache/uniffle/coordinator/strategy/assignment/PartitionBalanceAssignmentStrategy.java
+++ 
b/coordinator/src/main/java/org/apache/uniffle/coordinator/strategy/assignment/PartitionBalanceAssignmentStrategy.java
@@ -135,7 +135,7 @@ public class PartitionBalanceAssignmentStrategy extends 
AbstractAssignmentStrate
           });
 
       if (nodes.isEmpty() || nodes.size() < replica) {
-        throw new RssException("There isn't enough shuffle servers");
+        throw new RssException("There aren't enough shuffle servers");
       }
 
       final int assignmentMaxNum = clusterManager.getShuffleNodesMax();
diff --git 
a/integration-test/spark-common/src/test/java/org/apache/uniffle/test/RssShuffleManagerTest.java
 
b/integration-test/spark-common/src/test/java/org/apache/uniffle/test/RssShuffleManagerTest.java
index 72c47fe29..a3b09d142 100644
--- 
a/integration-test/spark-common/src/test/java/org/apache/uniffle/test/RssShuffleManagerTest.java
+++ 
b/integration-test/spark-common/src/test/java/org/apache/uniffle/test/RssShuffleManagerTest.java
@@ -153,7 +153,7 @@ public class RssShuffleManagerTest extends 
SparkIntegrationTestBase {
     conf.set(RssSparkConfig.RSS_DYNAMIC_CLIENT_CONF_ENABLED, 
enableDynamicClientConf);
     // configure storage type
     conf.set("spark." + RssClientConfig.RSS_STORAGE_TYPE, 
StorageType.MEMORY_LOCALFILE.name());
-    // restarting the coordinator may cause RssException: There isn't enough 
shuffle servers
+    // restarting the coordinator may cause RssException: There aren't enough 
shuffle servers
     // retry quickly (default is 65s interval)
     conf.set("spark." + RssClientConfig.RSS_CLIENT_ASSIGNMENT_RETRY_INTERVAL, 
"1000");
     conf.set("spark." + RssClientConfig.RSS_CLIENT_ASSIGNMENT_RETRY_TIMES, 
"10");

Reply via email to