This is an automated email from the ASF dual-hosted git repository.
dataroaring 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 3f86a1d2713 [fix](agg) Aggregating string types with null values may
result in incorrect result when compaction (#42512)
3f86a1d2713 is described below
commit 3f86a1d27135a59ecac47b43afac2020de2855d7
Author: Xin Liao <[email protected]>
AuthorDate: Mon Oct 28 10:24:17 2024 +0800
[fix](agg) Aggregating string types with null values may result in
incorrect result when compaction (#42512)
Follow-up for #42067.
---
be/src/vec/olap/vertical_block_reader.cpp | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/be/src/vec/olap/vertical_block_reader.cpp
b/be/src/vec/olap/vertical_block_reader.cpp
index ee2f52462f8..dd2c167324f 100644
--- a/be/src/vec/olap/vertical_block_reader.cpp
+++ b/be/src/vec/olap/vertical_block_reader.cpp
@@ -317,10 +317,10 @@ size_t VerticalBlockReader::_copy_agg_data() {
auto& dst_column = _stored_data_columns[idx];
if (_stored_has_variable_length_tag[idx]) {
//variable length type should replace ordered
+ dst_column->clear();
for (size_t i = 0; i < copy_size; i++) {
auto& ref = _stored_row_ref[i];
-
dst_column->replace_column_data(*ref.block->get_by_position(idx).column,
- ref.row_pos, i);
+
dst_column->insert_from(*ref.block->get_by_position(idx).column, ref.row_pos);
}
} else {
for (auto& it : _temp_ref_map) {
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
