chore: apply suggestions from CR

GreptimeTeam · Jan 4, 2024 · 817b650 · 817b650
1 parent aa2d4bb
commit 817b650
Showing 1 changed file with 2 additions and 7 deletions.
diff --git a/src/metric-engine/src/engine/alter.rs b/src/metric-engine/src/engine/alter.rs
@@ -29,14 +29,9 @@ impl MetricEngineInner {
         region_id: RegionId,
         request: RegionAlterRequest,
     ) -> Result<AffectedRows> {
-        let is_altering_logical_region = self
-            .state
-            .read()
-            .await
-            .physical_regions()
-            .contains_key(&region_id);
+        let is_altering_physical_region = self.is_physical_region(region_id).await;
 
-        let result = if is_altering_logical_region {
+        let result = if is_altering_physical_region {
             self.alter_physical_region(region_id, request).await
         } else {
             self.alter_logical_region(region_id, request).await