Skip to content

Commit

Permalink
fix comment
Browse files Browse the repository at this point in the history
  • Loading branch information
liaoxin01 committed Sep 24, 2024
1 parent be8de68 commit 9d103d8
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 7 deletions.
5 changes: 3 additions & 2 deletions be/src/cloud/cloud_delete_task.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -53,8 +53,9 @@ Status CloudDeleteTask::execute(CloudStorageEngine& engine, const TPushReq& requ
.tag("limit", config::max_tablet_version_num)
.tag("tablet_id", tablet->tablet_id());
return Status::Error<TOO_MANY_VERSION>(
"too many versions, versions={} tablet={}. Please reduce the frequency of loading "
"data.",
"too many versions, versions={} tablet={}. Please decrease the data loading "
"frequency or consider adjusting the max_tablet_version_num in be.conf to a higher "
"value.",
config::max_tablet_version_num, tablet->tablet_id());
}

Expand Down
3 changes: 2 additions & 1 deletion be/src/cloud/cloud_rowset_builder.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,8 @@ Status CloudRowsetBuilder::check_tablet_version_count() {
if (version_count > config::max_tablet_version_num) {
return Status::Error<TOO_MANY_VERSION>(
"failed to init rowset builder. version count: {}, exceed limit: {}, "
"tablet: {}. Please reduce the frequency of loading data.",
"tablet: {}. Please decrease the data loading frequency or consider adjusting the "
"max_tablet_version_num in be.conf to a higher value.",
version_count, config::max_tablet_version_num, _tablet->tablet_id());
}
return Status::OK();
Expand Down
6 changes: 4 additions & 2 deletions be/src/olap/push_handler.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -164,7 +164,8 @@ Status PushHandler::_do_streaming_ingestion(TabletSharedPtr tablet, const TPushR
if (tablet->exceed_version_limit(config::max_tablet_version_num)) {
return Status::Status::Error<TOO_MANY_VERSION>(
"failed to push data. version count: {}, exceed limit: {}, tablet: {}. Please "
"reduce the frequency of loading data.",
"decrease the data loading frequency or consider adjusting the "
"max_tablet_version_num in be.conf to a higher value.",
tablet->version_count(), config::max_tablet_version_num, tablet->tablet_id());
}

Expand All @@ -173,7 +174,8 @@ Status PushHandler::_do_streaming_ingestion(TabletSharedPtr tablet, const TPushR
config::tablet_meta_serialize_size_limit) {
return Status::Error<TOO_MANY_VERSION>(
"failed to init rowset builder. meta serialize size : {}, exceed limit: {}, "
"tablet: {}. Please reduce the frequency of loading data.",
"tablet: {}. Please decrease the data loading frequency or consider adjusting the "
"max_tablet_version_num in be.conf to a higher value.",
tablet->avg_rs_meta_serialize_size() * version_count,
config::tablet_meta_serialize_size_limit, tablet->tablet_id());
}
Expand Down
6 changes: 4 additions & 2 deletions be/src/olap/rowset_builder.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -165,7 +165,8 @@ Status RowsetBuilder::check_tablet_version_count() {
if (version_count > config::max_tablet_version_num) {
return Status::Error<TOO_MANY_VERSION>(
"failed to init rowset builder. version count: {}, exceed limit: {}, "
"tablet: {}. Please reduce the frequency of loading data.",
"tablet: {}. Please decrease the data loading frequency or consider adjusting the "
"max_tablet_version_num in be.conf to a higher value",
version_count, config::max_tablet_version_num, _tablet->tablet_id());
}
return Status::OK();
Expand Down Expand Up @@ -200,7 +201,8 @@ Status RowsetBuilder::init() {
config::tablet_meta_serialize_size_limit) {
return Status::Error<TOO_MANY_VERSION>(
"failed to init rowset builder. meta serialize size : {}, exceed limit: {}, "
"tablet: {}. Please reduce the frequency of loading data.",
"tablet: {}. Please decrease the data loading frequency or consider adjusting the "
"max_tablet_version_num in be.conf to a higher value",
tablet()->avg_rs_meta_serialize_size() * version_count,
config::tablet_meta_serialize_size_limit, _tablet->tablet_id());
}
Expand Down

0 comments on commit 9d103d8

Please sign in to comment.