From 74b007a630083e7afbe98113020c34115075796a Mon Sep 17 00:00:00 2001
From: Kaijie Chen <chenkaijie@selectdb.com>
Date: Tue, 25 Feb 2025 17:31:14 +0800
Subject: [PATCH] [improve](load) improve error message in commit

---
 .../apache/doris/transaction/DatabaseTransactionMgr.java    | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/fe/fe-core/src/main/java/org/apache/doris/transaction/DatabaseTransactionMgr.java b/fe/fe-core/src/main/java/org/apache/doris/transaction/DatabaseTransactionMgr.java
index a4dcd877ac3f73..cc2a4b1a90f20c 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/transaction/DatabaseTransactionMgr.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/transaction/DatabaseTransactionMgr.java
@@ -635,7 +635,7 @@ private void checkCommitStatus(List<Table> tableList, TransactionState transacti
 
                             String errMsg = String.format("Failed to commit txn %s, cause tablet %s succ replica num %s"
                                     + " < load required replica num %s. table %s, partition: [ id=%s, commit version %s"
-                                    + ", visible version %s ], this tablet detail: %s",
+                                    + ", visible version %s ], this tablet detail: %s. Please try again later.",
                                     transactionId, tablet.getId(), successReplicaNum, loadRequiredReplicaNum, tableId,
                                     partition.getId(), partition.getCommittedVersion(), partition.getVisibleVersion(),
                                     writeDetail);
@@ -659,14 +659,14 @@ private String getTabletWriteDetail(List<Replica> tabletSuccReplicas, List<Repli
                                     .collect(Collectors.toList())));
         }
         if (!tabletWriteFailedReplicas.isEmpty()) {
-            writeDetail += String.format("%s replicas write data failed: { %s }; ",
+            writeDetail += String.format("%s replicas write data failed: { %s }, please check BE log for details; ",
                     tabletWriteFailedReplicas.size(), Joiner.on(", ").join(
                             tabletWriteFailedReplicas.stream().map(replica -> replica.toStringSimple(true))
                                     .collect(Collectors.toList())));
         }
         if (!tabletVersionFailedReplicas.isEmpty()) {
             writeDetail += String.format("%s replicas write data succ but miss previous "
-                            + "version: { %s }.",
+                            + "version: { %s }, please check output of SHOW TABLET for details",
                     tabletVersionFailedReplicas.size(), Joiner.on(",").join(
                             tabletVersionFailedReplicas.stream().map(replica -> replica.toStringSimple(true))
                                     .collect(Collectors.toList())));