This is an automated email from the ASF dual-hosted git repository.
dataroaring pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new de6c7a792e9 [fix](chore) update dcheck to avoid core during stress
test (#28895)
de6c7a792e9 is described below
commit de6c7a792e95ea8d12b0f20677d1ee567bbec81a
Author: zhannngchen <48427519+zhannngc...@users.noreply.github.com>
AuthorDate: Sat Dec 23 18:49:57 2023 +0800
[fix](chore) update dcheck to avoid core during stress test (#28895)
---
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 47fd726b375..a5f34633d79 100644
--- a/be/src/olap/tablet.cpp
+++ b/be/src/olap/tablet.cpp
@@ -3016,7 +3016,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: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
