[doris] 13/31: [chore](compaction) Do not print the stack trace when the compaction task already exists (#25597)

[kxiao]

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

kxiao pushed a commit to branch branch-2.0
in repository https://gitbox.apache.org/repos/asf/doris.git

commit 3f5dfbe5ccf7ed554d442c4b75fd9713fa0b673a
Author: Xiaocc <598887...@qq.com>
AuthorDate: Wed Oct 18 21:44:17 2023 +0800

    [chore](compaction) Do not print the stack trace when the compaction task 
already exists (#25597)
---
 be/src/olap/olap_server.cpp | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/be/src/olap/olap_server.cpp b/be/src/olap/olap_server.cpp
index 401495bed2b..ac6d9326b86 100644
--- a/be/src/olap/olap_server.cpp
+++ b/be/src/olap/olap_server.cpp
@@ -739,15 +739,15 @@ Status 
StorageEngine::_submit_single_replica_compaction_task(TabletSharedPtr tab
     bool already_exist =
             _push_tablet_into_submitted_compaction(tablet, 
CompactionType::CUMULATIVE_COMPACTION);
     if (already_exist) {
-        return Status::AlreadyExist("compaction task has already been 
submitted, tablet_id={}",
-                                    tablet->tablet_id());
+        return Status::AlreadyExist<false>(
+                "compaction task has already been submitted, tablet_id={}", 
tablet->tablet_id());
     }
 
     already_exist = _push_tablet_into_submitted_compaction(tablet, 
CompactionType::BASE_COMPACTION);
     if (already_exist) {
         _pop_tablet_from_submitted_compaction(tablet, 
CompactionType::CUMULATIVE_COMPACTION);
-        return Status::AlreadyExist("compaction task has already been 
submitted, tablet_id={}",
-                                    tablet->tablet_id());
+        return Status::AlreadyExist<false>(
+                "compaction task has already been submitted, tablet_id={}", 
tablet->tablet_id());
     }
     Status st = tablet->prepare_single_replica_compaction(tablet, 
compaction_type);
     auto clean_single_replica_compaction = [tablet, this]() {
@@ -945,7 +945,7 @@ Status 
StorageEngine::_submit_compaction_task(TabletSharedPtr tablet,
     }
     bool already_exist = _push_tablet_into_submitted_compaction(tablet, 
compaction_type);
     if (already_exist) {
-        return Status::AlreadyExist(
+        return Status::AlreadyExist<false>(
                 "compaction task has already been submitted, tablet_id={}, 
compaction_type={}.",
                 tablet->tablet_id(), compaction_type);
     }


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