This is an automated email from the ASF dual-hosted git repository.
dataroaring pushed a commit to branch compaction_opt
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/compaction_opt by this push:
new f251f8ae77 [bugfix](vertical_compaction) fix dup&agg model has no
delete_sign (#14297)
f251f8ae77 is described below
commit f251f8ae774a005733f9b55f1687497cbafefa55
Author: yixiutt <102007456+yixi...@users.noreply.github.com>
AuthorDate: Tue Nov 15 22:23:38 2022 +0800
[bugfix](vertical_compaction) fix dup&agg model has no delete_sign (#14297)
---
be/src/olap/merger.cpp | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/be/src/olap/merger.cpp b/be/src/olap/merger.cpp
index f537794994..d46c094bc0 100644
--- a/be/src/olap/merger.cpp
+++ b/be/src/olap/merger.cpp
@@ -207,7 +207,9 @@ void Merger::vertical_split_columns(TabletSchemaSPtr
tablet_schema,
key_columns.emplace_back(sequence_col_idx);
}
delete_sign_idx = tablet_schema->field_index(DELETE_SIGN);
- key_columns.emplace_back(delete_sign_idx);
+ if (delete_sign_idx != -1) {
+ key_columns.emplace_back(delete_sign_idx);
+ }
}
VLOG_NOTICE << "sequence_col_idx=" << sequence_col_idx
<< ", delete_sign_idx=" << delete_sign_idx;
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
