Skip to content

Commit

Permalink
rm version
Browse files Browse the repository at this point in the history
  • Loading branch information
TangSiyang2001 committed Aug 9, 2023
1 parent af95f30 commit 95e4a24
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
4 changes: 1 addition & 3 deletions be/src/olap/delete_handler.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -63,8 +63,6 @@ Status DeleteHandler::generate_delete_predicate(const TabletSchema& schema,
}
}

// write version to distinguish against the former version of delete sub predicate
del_pred->set_sub_pred_version(2);
// Store delete condition
for (const TCondition& condition : conditions) {
if (condition.condition_values.size() > 1) {
Expand Down Expand Up @@ -300,7 +298,7 @@ Status DeleteHandler::init(TabletSchemaSPtr tablet_schema,
auto& delete_condition = delete_pred->delete_predicate();
DeleteConditions temp;
temp.filter_version = delete_pred->version().first;
if (delete_condition.has_sub_pred_version() && delete_condition.sub_pred_version() == 2) {
if (delete_condition.sub_predicates_size() == 0) {
_parse_column_pred(delete_pred_related_schema, delete_condition.sub_predicates_v2(),
&temp);
} else {
Expand Down
1 change: 0 additions & 1 deletion gensrc/proto/olap_file.proto
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,6 @@ message DeletePredicatePB {
repeated string sub_predicates = 2;
repeated InPredicatePB in_predicates = 3;
repeated DeleteSubPredicatePB sub_predicates_v2 = 4;
optional int32 sub_pred_version = 5; // if is set to 2, use sub_predicates_v2, is not set, use sub_predicates
}

message DeleteSubPredicatePB {
Expand Down

0 comments on commit 95e4a24

Please sign in to comment.