update

Yukang-Lian · Jul 7, 2023 · be20977 · be20977
1 parent bebf967
commit be20977
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 8 deletions.
diff --git a/be/src/olap/tablet.cpp b/be/src/olap/tablet.cpp
@@ -3240,13 +3240,6 @@ Status Tablet::commit_phase_update_delete_bitmap(
               << ", rowset_ids to del: " << rowset_ids_to_del.size()
               << ", cur max_version: " << cur_version << ", transaction_id: " << txn_id
               << ", cost: " << watch.get_elapse_time_us() << "(us), total rows: " << total_rows;
-
-    for (auto iter = delete_bitmap->delete_bitmap.begin();
-         iter != delete_bitmap->delete_bitmap.end(); ++iter) {
-        delete_bitmap->merge({std::get<0>(iter->first), std::get<1>(iter->first), cur_version},
-                             iter->second);
-        delete_bitmap->delete_bitmap.erase({std::get<0>(iter->first), std::get<1>(iter->first), 0});
-    }
     pre_rowset_ids = cur_rowset_ids;
     return Status::OK();
 }

diff --git a/be/src/olap/tablet_meta.cpp b/be/src/olap/tablet_meta.cpp
@@ -988,7 +988,7 @@ void DeleteBitmap::subset(const BitmapKey& start, const BitmapKey& end,
     roaring::Roaring roaring;
     DCHECK(start < end);
     std::shared_lock l(lock);
-    for (auto it = delete_bitmap.upper_bound(start); it != delete_bitmap.end(); ++it) {
+    for (auto it = delete_bitmap.lower_bound(start); it != delete_bitmap.end(); ++it) {
         auto& [k, bm] = *it;
         if (k >= end) {
             break;