weizuo93 commented on a change in pull request #5651:
URL: https://github.com/apache/incubator-doris/pull/5651#discussion_r613201246
##########
File path: be/src/olap/tablet.cpp
##########
@@ -88,6 +88,10 @@ OLAPStatus Tablet::_init_once_action() {
<< ", version_size=" << _tablet_meta->version_count();
// init cumulative compaction policy by type
+ if (_cumulative_compaction_type == "") {
Review comment:
> Now _cumulative_compaction_policy will be created in
_calc_cumulative_compaction_score, is it needed to create it here initially?
I also konw that we don't need to
execute`create_cumulative_compaction_policy()`here, but UT will have some
problems if we remove it. Next, I will refacor UT.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
