You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by mo...@apache.org on 2022/10/20 15:23:42 UTC

[doris] branch master updated: [fix] Fix potential unhandled exception cause data inconsistency (#11029)

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 95f437c506 [fix] Fix potential unhandled exception cause data inconsistency (#11029)
95f437c506 is described below

commit 95f437c5065c3a2a307c9ccda9889bc3bf11a649
Author: TsukiokaKogane <cb...@gmail.com>
AuthorDate: Thu Oct 20 23:23:36 2022 +0800

    [fix] Fix potential unhandled exception cause data inconsistency (#11029)
    
    Co-authored-by: TsukiokaKogane <cb...@gamil.com>
---
 .../java/org/apache/doris/transaction/DatabaseTransactionMgr.java   | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

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 210684fea9..843df334ee 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
@@ -934,7 +934,11 @@ public class DatabaseTransactionMgr {
                 LOG.debug("after set transaction {} to visible", transactionState);
             } finally {
                 writeUnlock();
-                transactionState.afterStateTransform(TransactionStatus.VISIBLE, txnOperated);
+                try {
+                    transactionState.afterStateTransform(TransactionStatus.VISIBLE, txnOperated);
+                } catch (UserException e) {
+                    LOG.warn("afterStateTransform txn {} failed. msg: {}", transactionId, e.getMessage());
+                }
             }
             updateCatalogAfterVisible(transactionState, db);
         } finally {


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