You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by mo...@apache.org on 2022/12/20 06:48:09 UTC
[doris] branch branch-1.2-lts updated: [enhancement](compaction) add delete_sign_index check before filter delete (#15189)
This is an automated email from the ASF dual-hosted git repository.
morningman pushed a commit to branch branch-1.2-lts
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-1.2-lts by this push:
new ea7b74ace8 [enhancement](compaction) add delete_sign_index check before filter delete (#15189)
ea7b74ace8 is described below
commit ea7b74ace8d62bdd81214eb86f7dba0aff91c5b5
Author: yixiutt <10...@users.noreply.github.com>
AuthorDate: Tue Dec 20 14:48:01 2022 +0800
[enhancement](compaction) add delete_sign_index check before filter delete (#15189)
cherry-pick #15190
---
be/src/vec/olap/block_reader.cpp | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/be/src/vec/olap/block_reader.cpp b/be/src/vec/olap/block_reader.cpp
index 655e59895b..5b56c5ef82 100644
--- a/be/src/vec/olap/block_reader.cpp
+++ b/be/src/vec/olap/block_reader.cpp
@@ -278,6 +278,11 @@ Status BlockReader::_unique_key_next_block(Block* block, MemPool* mem_pool, Obje
// do filter delete row in base compaction, only base compaction need to do the job
if (_filter_delete) {
int delete_sign_idx = _reader_context.tablet_schema->field_index(DELETE_SIGN);
+ if (delete_sign_idx <= 0 || delete_sign_idx >= target_columns.size()) {
+ LOG(INFO) << "delete sign idx " << delete_sign_idx
+ << " not invalid, skip filter delete in base compaction";
+ return Status::OK();
+ }
DCHECK(delete_sign_idx > 0);
MutableColumnPtr delete_filter_column = (*std::move(_delete_filter_column)).mutate();
reinterpret_cast<ColumnUInt8*>(delete_filter_column.get())->resize(target_block_row);
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org