(#5151) An already merged rowset should skip window check (#5152)

This commit is contained in:
wangbo
2020-12-26 11:40:44 +08:00
committed by GitHub
parent 16d52651f3
commit d9f1ffe9a0

View File

@ -427,8 +427,10 @@ void CumulativeCompactionPolicy::pick_candidate_rowsets(
int64_t now = UnixSeconds();
for (auto& it : rs_version_map) {
// find all rowset version greater than cumulative_point and skip the create time in skip_window_sec
if (it.first.first >= cumulative_point &&
(it.second->creation_time() + skip_window_sec < now)) {
if (it.first.first >= cumulative_point
&& ((it.second->creation_time() + skip_window_sec < now)
// this case means a rowset has been compacted before which is not a new published rowset, so it should participate compaction
|| (it.first.first != it.first.second))) {
candidate_rowsets->push_back(it.second);
}
}