[fix](create tablet) fix disk usage may change when chose disks (#30508)

apache · Jan 30, 2024 · 57ff037 · 57ff037
1 parent 6f30b96
commit 57ff037
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/be/src/olap/storage_engine.cpp b/be/src/olap/storage_engine.cpp
@@ -478,10 +478,12 @@ void StorageEngine::_get_candidate_stores(TStorageMedium::type storage_medium,
             if ((_available_storage_medium_type_count == 1 ||
                  data_dir->storage_medium() == storage_medium) &&
                 !data_dir->reach_capacity_limit(0)) {
+                double usage = data_dir->get_usage(0);
                 DirInfo dir_info;
                 dir_info.data_dir = data_dir;
+                dir_info.usage = usage;
                 dir_info.available_level = 0;
-                usages.push_back(data_dir->get_usage(0));
+                usages.push_back(usage);
                 dir_infos.push_back(dir_info);
             }
         }
@@ -512,7 +514,7 @@ void StorageEngine::_get_candidate_stores(TStorageMedium::type storage_medium,
         }
     }
     for (auto& dir_info : dir_infos) {
-        double usage = dir_info.data_dir->get_usage(0);
+        double usage = dir_info.usage;
         for (size_t i = 1; i < level_min_usages.size() && usage >= level_min_usages[i]; i++) {
             dir_info.available_level++;
         }

diff --git a/be/src/olap/storage_engine.h b/be/src/olap/storage_engine.h
@@ -539,6 +539,7 @@ class CreateTabletIdxCache : public LRUCachePolicy {
 struct DirInfo {
     DataDir* data_dir;
 
+    double usage = 0;
     int available_level = 0;
 
     bool operator<(const DirInfo& other) const {