This is an automated email from the ASF dual-hosted git repository.
yiguolei pushed a commit to branch branch-4.0
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-4.0 by this push:
new 91d7bcb8441 branch-4.0: [Chore](bitmap) change BitmapValue CHECK to
throw exception #59464 (#59491)
91d7bcb8441 is described below
commit 91d7bcb844194e4dc74852bbf499b2d4928ed355
Author: github-actions[bot]
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Wed Dec 31 11:58:58 2025 +0800
branch-4.0: [Chore](bitmap) change BitmapValue CHECK to throw exception
#59464 (#59491)
Cherry-picked from #59464
Co-authored-by: Pxl <[email protected]>
---
be/src/util/bitmap_value.h | 11 +++++++++--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/be/src/util/bitmap_value.h b/be/src/util/bitmap_value.h
index 06a18f8bb4f..dbdd1ff84c1 100644
--- a/be/src/util/bitmap_value.h
+++ b/be/src/util/bitmap_value.h
@@ -1977,12 +1977,19 @@ public:
++src;
uint8_t count = *src;
++src;
- CHECK(count <= SET_TYPE_THRESHOLD) << "bitmap value with incorrect
set count";
+ if (count > SET_TYPE_THRESHOLD) {
+ throw Exception(ErrorCode::INTERNAL_ERROR,
+ "bitmap value with incorrect set count, count:
{}", count);
+ }
_set.reserve(count);
for (uint8_t i = 0; i != count; ++i, src += sizeof(uint64_t)) {
_set.insert(decode_fixed64_le(reinterpret_cast<const
uint8_t*>(src)));
}
- CHECK_EQ(count, _set.size()) << "bitmap value with incorrect set
count";
+ if (_set.size() != count) {
+ throw Exception(ErrorCode::INTERNAL_ERROR,
+ "bitmap value with incorrect set count, count:
{}, set size: {}",
+ count, _set.size());
+ }
if (!config::enable_set_in_bitmap_value) {
_prepare_bitmap_for_write();
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
