[Bug](bitmap) clear set when bitmap fastunion (apache#37816)

## Proposed changes before:6 min 59.33 sec after :24.67 sec
dataroaring · Jul 15, 2024 · 01f00aa · 01f00aa
1 parent b8b36c6
commit 01f00aa
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/be/src/util/bitmap_value.h b/be/src/util/bitmap_value.h
@@ -1252,8 +1252,7 @@ class BitmapValue {
         std::vector<const detail::Roaring64Map*> bitmaps;
         std::vector<uint64_t> single_values;
         std::vector<const SetContainer<uint64_t>*> sets;
-        for (int i = 0; i < values.size(); ++i) {
-            auto* value = values[i];
+        for (const auto* value : values) {
             switch (value->_type) {
             case EMPTY:
                 break;
@@ -1280,7 +1279,9 @@ class BitmapValue {
                 _bitmap->add(_sv);
                 break;
             case BITMAP:
-                *_bitmap |= detail::Roaring64Map::fastunion(bitmaps.size(), bitmaps.data());
+                for (const auto* bitmap : bitmaps) {
+                    *_bitmap |= *bitmap;
+                }
                 break;
             case SET: {
                 *_bitmap = detail::Roaring64Map::fastunion(bitmaps.size(), bitmaps.data());
@@ -1315,6 +1316,7 @@ class BitmapValue {
                     _bitmap->add(v);
                 }
                 _type = BITMAP;
+                _set.clear();
                 break;
             case SET: {
                 break;