diff --git a/src/metric-engine/src/engine/alter.rs b/src/metric-engine/src/engine/alter.rs
index 0a321dc8cbd8..ac0c28b646f1 100644
--- a/src/metric-engine/src/engine/alter.rs
+++ b/src/metric-engine/src/engine/alter.rs
@@ -137,7 +137,7 @@ impl MetricEngineInner {
             .collect::<HashMap<_, _>>();
 
         // register columns to logical region
-        // we need to use modfied column metadata from physical region, since it may have been altered(especialy column id)
+        // we need to use modified column metadata from physical region, since it may have been altered(especially column id)
         for col in columns {
             if let Some(metadata) = after_alter_cols.get(&col.column_metadata.column_schema.name) {
                 self.metadata_region
diff --git a/src/metric-engine/src/engine/create.rs b/src/metric-engine/src/engine/create.rs
index 75cefcefc0a1..8440e3bd878f 100644
--- a/src/metric-engine/src/engine/create.rs
+++ b/src/metric-engine/src/engine/create.rs
@@ -278,11 +278,11 @@ impl MetricEngineInner {
         &self,
         data_region_id: RegionId,
         logical_region_id: RegionId,
-        mut new_columns: &mut [ColumnMetadata],
+        new_columns: &mut [ColumnMetadata],
     ) -> Result<()> {
         // alter data region
         self.data_region
-            .add_columns(data_region_id, &mut new_columns)
+            .add_columns(data_region_id, new_columns)
             .await?;
 
         // safety: previous step has checked this