This is an automated email from the ASF dual-hosted git repository.
kxiao pushed a commit to tag 2.0.15.1
in repository https://gitbox.apache.org/repos/asf/doris.git
commit 64bde86ec6028aed8f52d7eb5ac6c2ee01d2a982
Author: Xin Liao <liaoxin...@126.com>
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: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
