This is an automated email from the ASF dual-hosted git repository.
zhangchen pushed a commit to branch branch-2.0
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-2.0 by this push:
new 5384fe6b905 [branch-2.0](cherry-pick) update dcheck to avoid core
during stress test (#28895) (#29174)
5384fe6b905 is described below
commit 5384fe6b90556e134ae503a01b1a41bc3233f5b7
Author: zhannngchen <[email protected]>
AuthorDate: Wed Dec 27 21:43:28 2023 +0800
[branch-2.0](cherry-pick) update dcheck to avoid core during stress test
(#28895) (#29174)
---
be/src/olap/tablet.cpp | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/be/src/olap/tablet.cpp b/be/src/olap/tablet.cpp
index 611e32828bb..b74fe320799 100644
--- a/be/src/olap/tablet.cpp
+++ b/be/src/olap/tablet.cpp
@@ -3049,7 +3049,10 @@ Status
Tablet::calc_segment_delete_bitmap(RowsetSharedPtr rowset,
auto st = lookup_row_key(key, true, specified_rowsets, &loc,
dummy_version.first - 1,
segment_caches, &rowset_find);
bool expected_st = st.ok() || st.is<KEY_NOT_FOUND>() ||
st.is<KEY_ALREADY_EXISTS>();
- DCHECK(expected_st) << "unexpected error status while
lookup_row_key:" << st;
+ // It's a defensive DCHECK, we need to exclude some common errors
to avoid core-dump
+ // while stress test
+ DCHECK(expected_st || st.is<MEM_LIMIT_EXCEEDED>())
+ << "unexpected error status while lookup_row_key:" << st;
if (!expected_st) {
return st;
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
