You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by kx...@apache.org on 2023/10/22 16:58:00 UTC
[doris] 13/31: [chore](compaction) Do not print the stack trace when the compaction task already exists (#25597)
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 <59...@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-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org