[doris] 01/07: [fix](forward) fix MissingFormatArgumentException when failed to forward stmt to Master (#22142)

[morningman]

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

morningman pushed a commit to branch branch-1.2-lts
in repository https://gitbox.apache.org/repos/asf/doris.git

commit 31c54db5b6ca93c9f0b05a4da04b4bbddf61a95a
Author: Mingyu Chen <morning...@163.com>
AuthorDate: Wed Jul 26 09:00:04 2023 +0800

    [fix](forward) fix MissingFormatArgumentException when failed to forward 
stmt to Master (#22142)
---
 fe/fe-core/src/main/java/org/apache/doris/qe/MasterOpExecutor.java | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/fe/fe-core/src/main/java/org/apache/doris/qe/MasterOpExecutor.java 
b/fe/fe-core/src/main/java/org/apache/doris/qe/MasterOpExecutor.java
index 5ba69a51c2..d41edf7084 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/qe/MasterOpExecutor.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/qe/MasterOpExecutor.java
@@ -114,9 +114,9 @@ public class MasterOpExecutor {
             // may throw NullPointerException. add err msg
             throw new Exception("Failed to get master client.", e);
         }
-        final StringBuilder forwardMsg = new 
StringBuilder(String.format("forward to Master %s", thriftAddress));
+        final StringBuilder forwardMsg = new StringBuilder("forward to master 
FE %s" + thriftAddress.toString());
         if (!params.isSyncJournalOnly()) {
-            forwardMsg.append(", statement: %s").append(ctx.getStmtId());
+            forwardMsg.append(", statement id: ").append(ctx.getStmtId());
         }
         LOG.info(forwardMsg.toString());
 
@@ -128,7 +128,7 @@ public class MasterOpExecutor {
         } catch (TTransportException e) {
             // wrap the raw exception.
             forwardMsg.append(" : failed");
-            Exception exception = new 
ForwardToMasterException(String.format(forwardMsg.toString()), e);
+            Exception exception = new 
ForwardToMasterException(forwardMsg.toString(), e);
 
             boolean ok = ClientPool.frontendPool.reopen(client, 
thriftTimeoutMs);
             if (!ok) {


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