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

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


The following commit(s) were added to refs/heads/master by this push:
     new 422cb9d4c RATIS-2043. Change toString() of RaftConfigurationImpl 
(#1051)
422cb9d4c is described below

commit 422cb9d4cb3f6be8e3169797d11a3c2e666c138f
Author: Symious <[email protected]>
AuthorDate: Fri Mar 15 09:22:03 2024 +0800

    RATIS-2043. Change toString() of RaftConfigurationImpl (#1051)
---
 .../main/java/org/apache/ratis/server/impl/RaftConfigurationImpl.java   | 2 +-
 .../src/main/java/org/apache/ratis/server/impl/RaftServerImpl.java      | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/ratis-server/src/main/java/org/apache/ratis/server/impl/RaftConfigurationImpl.java
 
b/ratis-server/src/main/java/org/apache/ratis/server/impl/RaftConfigurationImpl.java
index d609264af..98f41020e 100644
--- 
a/ratis-server/src/main/java/org/apache/ratis/server/impl/RaftConfigurationImpl.java
+++ 
b/ratis-server/src/main/java/org/apache/ratis/server/impl/RaftConfigurationImpl.java
@@ -298,7 +298,7 @@ final class RaftConfigurationImpl implements 
RaftConfiguration {
 
   @Override
   public String toString() {
-    return logEntryIndex + ": " + conf + ", old=" + oldConf;
+    return "conf: {index: " + logEntryIndex + ", cur=" + conf + ", old=" + 
oldConf + "}";
   }
 
   boolean hasNoChange(Collection<RaftPeer> newMembers, Collection<RaftPeer> 
newListeners) {
diff --git 
a/ratis-server/src/main/java/org/apache/ratis/server/impl/RaftServerImpl.java 
b/ratis-server/src/main/java/org/apache/ratis/server/impl/RaftServerImpl.java
index e8aeb6678..23cab70d3 100644
--- 
a/ratis-server/src/main/java/org/apache/ratis/server/impl/RaftServerImpl.java
+++ 
b/ratis-server/src/main/java/org/apache/ratis/server/impl/RaftServerImpl.java
@@ -1210,7 +1210,7 @@ class RaftServerImpl implements RaftServer.Division,
 
       if (!conf.isHighestPriority(request.getNewLeader())) {
         String msg = getMemberId() + " refused to transfer leadership to peer 
" + request.getNewLeader() +
-            " as it does not has highest priority " + conf;
+            " as it does not has highest priority in " + conf;
         return logAndReturnTransferLeadershipFail(request, msg);
       }
 

Reply via email to