Add method client_service::is_prepared_xa()

dbsim/db_client_service.hpp

@@ -82,6 +82,11 @@ namespace db
             return false;
         }
 
+        bool is_prepared_xa() override
+        {
+            return false;
+        }
+
         bool is_xa_rollback() override
         {
             return false;

include/wsrep/client_service.hpp

@@ -194,6 +194,16 @@ namespace wsrep
          */
         virtual bool is_explicit_xa() = 0;
 
+        /**
+         * Returns true if the client has an ongoing XA transaction
+         * in prepared state.
+         * Notice: one could simply check if wsrep::transaction is
+         * in s_prepared state. However, wsrep::transaction does not
+         * transition to prepared state for read-only / empty
+         * transactions.
+         */
+        virtual bool is_prepared_xa() = 0;
+
         /**
          * Returns true if the currently executing command is
          * a rollback for XA. This is used to avoid setting a

src/client_state.cpp

@@ -68,8 +68,7 @@ void wsrep::client_state::close()
     keep_command_error_ = false;
     lock.unlock();
     if (transaction_.active() &&
-        (mode_ != m_local ||
+        (mode_ != m_local || !client_service_.is_prepared_xa()))
-         transaction_.state() != wsrep::transaction::s_prepared))
     {
         client_service_.bf_rollback();
         transaction_.after_statement();

test/mock_client_state.hpp

@@ -176,6 +176,11 @@ namespace wsrep
             return false;
         }
 
+        bool is_prepared_xa() WSREP_OVERRIDE
+        {
+            return false;
+        }
+
         bool is_xa_rollback() WSREP_OVERRIDE
         {
             return false;