BiteTheDDDDt commented on code in PR #9804:
URL: https://github.com/apache/incubator-doris/pull/9804#discussion_r883230780
##########
be/src/olap/tablet.cpp:
##########
@@ -839,10 +839,53 @@ void Tablet::calculate_cumulative_point() {
if (ret_cumulative_point == K_INVALID_CUMULATIVE_POINT) {
return;
}
-
set_cumulative_layer_point(ret_cumulative_point);
}
+//1.小版本rowset
+//2.不改变cp，cp值的更新有cc任务来执行
+//3.找到最大的，连续小版本rowset
+Status Tablet::pick_samll_verson_rowsets(std::vector<RowsetSharedPtr>*
input_rowsets) {
+ int max_series_num = 1000;
+ int max_rows = config::small_version_max_rows;
+ if (max_rows <= 0) return Status::OK();
+ std::vector<std::vector<RowsetSharedPtr>>
samll_version_rowsets(max_series_num);
+ int idx = 0;
+ bool is_bad = false;
+ if (tablet_state() == TABLET_RUNNING) {
+ for (auto& rs : _rs_version_map) {
+ bool is_delete = version_for_delete_predicate(rs.first);
+ if (rs.first.first > cumulative_layer_point()) {
Review Comment:
Maybe you can use more `guard clauses` on here.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
