fix: replace if else with match

GreptimeTeam · Apr 11, 2024 · 47e7606 · 47e7606
1 parent b937ea4
commit 47e7606
Showing 1 changed file with 22 additions and 17 deletions.
diff --git a/src/servers/src/http.rs b/src/servers/src/http.rs
@@ -705,23 +705,28 @@ impl HttpServer {
         strict_mode: bool,
     ) -> Router<S> {
         let mut router = Router::new().route("/read", routing::post(prom_store::remote_read));
-        if prom_store_with_metric_engine && strict_mode {
-            router = router.route("/write", routing::post(prom_store::remote_write));
-        } else if prom_store_with_metric_engine && !strict_mode {
-            router = router.route(
-                "/write",
-                routing::post(prom_store::remote_write_without_strict_mode),
-            );
-        } else if !prom_store_with_metric_engine && strict_mode {
-            router = router.route(
-                "/write",
-                routing::post(prom_store::route_write_without_metric_engine),
-            );
-        } else {
-            router = router.route(
-                "/write",
-                routing::post(prom_store::route_write_without_metric_engine_and_strict_mode),
-            );
+        match (prom_store_with_metric_engine, strict_mode) {
+            (true, true) => {
+                router = router.route("/write", routing::post(prom_store::remote_write))
+            }
+            (true, false) => {
+                router = router.route(
+                    "/write",
+                    routing::post(prom_store::remote_write_without_strict_mode),
+                )
+            }
+            (false, true) => {
+                router = router.route(
+                    "/write",
+                    routing::post(prom_store::route_write_without_metric_engine),
+                )
+            }
+            (false, false) => {
+                router = router.route(
+                    "/write",
+                    routing::post(prom_store::route_write_without_metric_engine_and_strict_mode),
+                )
+            }
         }
         router.with_state(prom_handler)
     }