This is an automated email from the ASF dual-hosted git repository.
yiguolei 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 a83ac5125b3 [fix](be) Fix ColumnComplexTest.GetDataAtTest ut failure
(#64185)
a83ac5125b3 is described below
commit a83ac5125b34253cf00be7691e20b1e60bd2eb3c
Author: heguanhui <[email protected]>
AuthorDate: Tue Jun 9 11:49:15 2026 +0800
[fix](be) Fix ColumnComplexTest.GetDataAtTest ut failure (#64185)
### What problem does this PR solve?
Issue Number: close #64184
Related PR: #
Problem Summary: The BE unit test `ColumnComplexTest.GetDataAtTest`
fails because `get_data_at()` for `ColumnBitmap`, `ColumnHll`, and
`ColumnQuantileState` returns raw binary data that includes
uninitialized padding bytes, causing comparison failures even when the
actual data values are equal.
The fix replaces `get_data_at()` comparisons with
`get_element().to_string()` for bitmap/hll columns and `operator[]` for
quantile_state column, which compare logical values instead of raw
binary representations.
### Release note
None
### Check List (For Author)
- Test
- [x] Unit Test
- Behavior changed:
- [x] No.
- Does this need documentation?
- [x] No.
Co-authored-by: root <root@DESKTOP-3AF37B>
---
be/test/core/block/column_complex_test.cpp | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/be/test/core/block/column_complex_test.cpp
b/be/test/core/block/column_complex_test.cpp
index ec8fd78d105..3a2629474f9 100644
--- a/be/test/core/block/column_complex_test.cpp
+++ b/be/test/core/block/column_complex_test.cpp
@@ -76,8 +76,10 @@ TEST(ColumnComplexTest, GetDataAtTest) {
ASSERT_EQ(column_bitmap->size(), 1);
ASSERT_EQ(column_bitmap_verify->size(), 1);
- ASSERT_EQ(column_bitmap->get_data_at(0),
column_bitmap_verify->get_data_at(0));
- ASSERT_EQ(column_hll->get_data_at(0), column_hll_verify->get_data_at(0));
+ ASSERT_EQ(column_bitmap->get_element(0).to_string(),
+ column_bitmap_verify->get_element(0).to_string());
+ ASSERT_EQ(column_hll->get_element(0).to_string(),
+ column_hll_verify->get_element(0).to_string());
ASSERT_EQ(column_quantile_state->operator[](0),
column_quantile_state_verify->operator[](0));
std::cout << "3. test insert_default/insert_value empty success" <<
std::endl;
@@ -117,7 +119,7 @@ TEST(ColumnComplexTest, GetDataAtTest) {
ASSERT_EQ(column_bitmap->get_element(1).to_string(),
bitmap_verify_data[0].to_string());
ASSERT_EQ(column_hll->get_element(1).to_string(),
column_hll_verify->get_element(0).to_string());
- ASSERT_EQ(column_quantile_state->get_data_at(1),
column_quantile_state_verify->get_data_at(0));
+ ASSERT_EQ(column_quantile_state->operator[](1),
column_quantile_state_verify->operator[](0));
std::cout << "4. test insert/update/get_element data success" << std::endl;
// insert data from column idx 1
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
