This is an automated email from the ASF dual-hosted git repository.

gavinchou pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
     new 75557a5bb9c [fix](cloud-mow) tablet_id should use int64_t to avoid 
truncation problem when removing old delete bitmap (#43956)
75557a5bb9c is described below

commit 75557a5bb9c107db603682e93f67612bfe429e07
Author: huanghaibin <huanghai...@selectdb.com>
AuthorDate: Thu Nov 14 23:33:26 2024 +0800

    [fix](cloud-mow) tablet_id should use int64_t to avoid truncation problem 
when removing old delete bitmap (#43956)
    
    tablet_id is int64_t, use int32_t may have truncation problem, which
    will lead to send wrong tablet_id to ms, and do nothing on remove old
    delete bitmap
---
 be/src/olap/tablet_meta.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/be/src/olap/tablet_meta.cpp b/be/src/olap/tablet_meta.cpp
index 13bbdaa9389..4005c818bc5 100644
--- a/be/src/olap/tablet_meta.cpp
+++ b/be/src/olap/tablet_meta.cpp
@@ -1198,7 +1198,7 @@ void 
DeleteBitmap::remove_stale_delete_bitmap_from_queue(const std::vector<std::
     std::shared_lock l(stale_delete_bitmap_lock);
     //<rowset_id, start_version, end_version>
     std::vector<std::tuple<std::string, uint64_t, uint64_t>> to_delete;
-    auto tablet_id = -1;
+    int64_t tablet_id = -1;
     for (auto& version_str : vector) {
         auto it = _stale_delete_bitmap.find(version_str);
         if (it != _stale_delete_bitmap.end()) {


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to