Skip to content

Fix be may core dump when linked schema change #4079

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
yangzhg merged 3 commits into from
Jul 15, 2020
Merged

Fix be may core dump when linked schema change #4079

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
4826910
fix a core
yangzhg Jul 10, 2020
1259bfe
Update be/src/olap/rowset/segment_group.cpp
yangzhg Jul 10, 2020
aae67a9
Update segment_group.cpp
yangzhg Jul 13, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion be/src/olap/rowset/segment_group.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -256,7 +256,10 @@ OLAPStatus SegmentGroup::add_zone_maps_for_linked_schema_change(

int zonemap_col_num = get_num_zone_map_columns();
CHECK(zonemap_col_num <= schema_mapping.size()) << zonemap_col_num << " vs. " << schema_mapping.size();
for (size_t i = 0; i < zonemap_col_num; ++i) {
// if upgrade from 0.11 the zone map not contain the value column, so zone_map_fields.size() is less
// than zonemap_col_num, this may be cause core dump at access zone_map_fields by index in the following code,
// so check the zone_map_fields.size() to set the actual zone map fields
for (size_t i = 0; i < zonemap_col_num && i < zone_map_fields.size(); ++i) {
const TabletColumn& column = _schema->column(i);

WrapperField* first = WrapperField::create(column);
Expand Down