Skip to content

Commit

Permalink
[fix](inverted_index) donot use int32_t for index id to avoid overflow
Browse files Browse the repository at this point in the history
g On branch index_id_overflow
  • Loading branch information
dataroaring committed May 20, 2024
1 parent 212ce7c commit 93b9c39
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion be/src/olap/task/index_builder.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ Status IndexBuilder::update_inverted_index_info() {
LOG(INFO) << "begin to update_inverted_index_info, tablet=" << _tablet->tablet_id()
<< ", is_drop_op=" << _is_drop_op;
// index ids that will not be linked
std::set<int32_t> without_index_uids;
std::set<int64_t> without_index_uids;
_output_rowsets.reserve(_input_rowsets.size());
_pending_rs_guards.reserve(_input_rowsets.size());
for (auto&& input_rowset : _input_rowsets) {
Expand Down
2 changes: 1 addition & 1 deletion be/src/olap/task/index_builder.h
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ class IndexBuilder {
std::vector<TColumn> _columns;
std::vector<doris::TOlapTableIndex> _alter_inverted_indexes;
bool _is_drop_op;
std::set<int32_t> _alter_index_ids;
std::set<int64_t> _alter_index_ids;
std::vector<RowsetSharedPtr> _input_rowsets;
std::vector<RowsetSharedPtr> _output_rowsets;
std::vector<PendingRowsetGuard> _pending_rs_guards;
Expand Down

0 comments on commit 93b9c39

Please sign in to comment.