You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by da...@apache.org on 2022/11/15 14:23:46 UTC
[doris] branch compaction_opt updated: [bugfix](vertical_compaction) fix dup&agg model has no delete_sign (#14297)
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 <10...@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-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org