(kudu) branch master updated: Fix a mistake of using function 'Substitute'

[alexey]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 7b0a7d3ec Fix a mistake of using function 'Substitute'
7b0a7d3ec is described below

commit 7b0a7d3ec7d17b422f3e736da6576eea2e035236
Author: xinghuayu007 <1450306...@qq.com>
AuthorDate: Fri Aug 2 16:30:13 2024 +0800

    Fix a mistake of using function 'Substitute'
    
    Change-Id: Ia8ad5d842c8c7a5f4f09793f91f323b8d73954c2
    Reviewed-on: http://gerrit.cloudera.org:8080/21635
    Reviewed-by: Mahesh Reddy <mre...@cloudera.com>
    Tested-by: Alexey Serbin <ale...@apache.org>
    Reviewed-by: Alexey Serbin <ale...@apache.org>
---
 src/kudu/master/catalog_manager.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/kudu/master/catalog_manager.cc 
b/src/kudu/master/catalog_manager.cc
index c62145f3f..6ea1504bd 100644
--- a/src/kudu/master/catalog_manager.cc
+++ b/src/kudu/master/catalog_manager.cc
@@ -6222,7 +6222,7 @@ Status CatalogManager::SelectReplicasForTablet(const 
PlacementPolicy& policy,
       return Status::InvalidArgument(
           Substitute("need at least $0 out of $1 replicas to form a Raft 
quorum, "
                      "but only $2 tablet servers are online",
-                     consensus::MajoritySize(nreplicas), policy.ts_num()));
+                     consensus::MajoritySize(nreplicas), nreplicas, 
policy.ts_num()));
     }
     nreplicas = policy.ts_num();
   }