fix: save code in debug_assert! (GreptimeTeam#3137)

fix: save code in debug_assert!
evenyag · Jan 11, 2024 · 312e8e8 · 312e8e8
1 parent 29a7f30
commit 312e8e8
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/src/meta-srv/src/procedure/region_migration/update_metadata/downgrade_leader_region.rs b/src/meta-srv/src/procedure/region_migration/update_metadata/downgrade_leader_region.rs
@@ -60,13 +60,13 @@ impl UpdateMetadata {
             .await
             .context(error::TableMetadataManagerSnafu)
         {
-            debug_assert!(ctx.remove_table_route_value());
+            ctx.remove_table_route_value();
             return error::RetryLaterSnafu {
                 reason: format!("Failed to update the table route during the downgrading leader region, error: {err}")
             }.fail();
         }
 
-        debug_assert!(ctx.remove_table_route_value());
+        ctx.remove_table_route_value();
 
         Ok(())
     }

diff --git a/src/meta-srv/src/procedure/region_migration/update_metadata/rollback_downgraded_region.rs b/src/meta-srv/src/procedure/region_migration/update_metadata/rollback_downgraded_region.rs
@@ -44,13 +44,13 @@ impl UpdateMetadata {
             .await
             .context(error::TableMetadataManagerSnafu)
         {
-            debug_assert!(ctx.remove_table_route_value());
+            ctx.remove_table_route_value();
             return error::RetryLaterSnafu {
                 reason: format!("Failed to update the table route during the rollback downgraded leader region, error: {err}")
             }.fail();
         }
 
-        debug_assert!(ctx.remove_table_route_value());
+        ctx.remove_table_route_value();
 
         Ok(())
     }

diff --git a/src/meta-srv/src/procedure/region_migration/update_metadata/upgrade_candidate_region.rs b/src/meta-srv/src/procedure/region_migration/update_metadata/upgrade_candidate_region.rs
@@ -167,13 +167,13 @@ impl UpdateMetadata {
             .await
             .context(error::TableMetadataManagerSnafu)
         {
-            debug_assert!(ctx.remove_table_route_value());
+            ctx.remove_table_route_value();
             return error::RetryLaterSnafu {
                     reason: format!("Failed to update the table route during the upgrading candidate region, error: {err}")
                 }.fail();
         };
 
-        debug_assert!(ctx.remove_table_route_value());
+        ctx.remove_table_route_value();
         // Consumes the guard.
         ctx.volatile_ctx.opening_region_guard.take();