From 130b3599bc472ab146eae572dbce5c428a2f9b13 Mon Sep 17 00:00:00 2001
From: Kang <kxiao.tiger@gmail.com>
Date: Thu, 9 Feb 2023 19:48:23 +0800
Subject: [PATCH] [Improvement](writer) make DeltaWriter close idempotent to be
 more robust (#16558)

return `Status::OK()` instead of `Status::Error<ALREADY_CLOSED>()` for close() in `DeltaWriter` if it's already closed.
---
 be/src/olap/delta_writer.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/be/src/olap/delta_writer.cpp b/be/src/olap/delta_writer.cpp
index 1f3cfe40e4..ee0b89570a 100644
--- a/be/src/olap/delta_writer.cpp
+++ b/be/src/olap/delta_writer.cpp
@@ -293,7 +293,7 @@ Status DeltaWriter::close() {
     if (_is_closed) {
         LOG(WARNING) << "close after closed tablet_id=" << _req.tablet_id
                      << " load_id=" << _req.load_id << " txn_id=" << _req.txn_id;
-        return Status::Error<ALREADY_CLOSED>();
+        return Status::OK();
     }
 
     auto s = _flush_memtable_async();