fixup

Signed-off-by: tison <[email protected]>
GreptimeTeam · Apr 8, 2024 · d2eed19 · d2eed19
1 parent 882b9cf
commit d2eed19
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 10 deletions.
diff --git a/src/meta-srv/src/service/admin.rs b/src/meta-srv/src/service/admin.rs
@@ -98,10 +98,12 @@ pub fn make_admin_service(meta_srv: MetaSrv) -> Admin {
     };
     let router = router.route("/region-migration", handler);
 
-    let router = router
-        .route("/maintenance", maintenance::MaintenanceHandler {
+    let router = router.route(
+        "/maintenance",
+        maintenance::MaintenanceHandler {
             kv_backend: meta_srv.kv_backend().clone(),
-        });
+        },
+    );
     let router = Router::nest("/admin", router);
 
     Admin::new(router)
@@ -302,7 +304,11 @@ mod tests {
         let router = Router::nest("/test_root", router);
 
         let res = router
-            .call("/test_root/test_node", http::Method::GET, HashMap::default())
+            .call(
+                "/test_root/test_node",
+                http::Method::GET,
+                HashMap::default(),
+            )
             .await
             .unwrap();
 
@@ -314,7 +320,11 @@ mod tests {
         let router = Router::new();
 
         let res = router
-            .call("/test_root/test_node", http::Method::GET, HashMap::default())
+            .call(
+                "/test_root/test_node",
+                http::Method::GET,
+                HashMap::default(),
+            )
             .await
             .unwrap();
 
@@ -328,7 +338,11 @@ mod tests {
         let router = Router::nest("/test_root", router);
 
         let res = router
-            .call("/test_root/test_node", http::Method::GET, HashMap::default())
+            .call(
+                "/test_root/test_node",
+                http::Method::GET,
+                HashMap::default(),
+            )
             .await
             .unwrap();
 

diff --git a/src/meta-srv/src/service/admin/heartbeat.rs b/src/meta-srv/src/service/admin/heartbeat.rs
@@ -32,8 +32,8 @@ pub struct HeartBeatHandler {
 impl HttpHandler for HeartBeatHandler {
     async fn handle(
         &self,
-        _: http::Method,
         path: &str,
+        _: http::Method,
         params: &HashMap<String, String>,
     ) -> Result<http::Response<String>> {
         if path.ends_with("/help") {

diff --git a/src/meta-srv/src/service/admin/maintenance.rs b/src/meta-srv/src/service/admin/maintenance.rs
@@ -21,7 +21,10 @@ use snafu::{OptionExt, ResultExt};
 use tonic::codegen::http;
 use tonic::codegen::http::Response;
 
-use crate::error::{InvalidHttpBodySnafu, KvBackendSnafu, MissingRequiredParameterSnafu, ParseBoolSnafu, UnsupportedSnafu};
+use crate::error::{
+    InvalidHttpBodySnafu, KvBackendSnafu, MissingRequiredParameterSnafu, ParseBoolSnafu,
+    UnsupportedSnafu,
+};
 use crate::service::admin::HttpHandler;
 
 #[derive(Clone)]
@@ -89,16 +92,17 @@ impl MaintenanceHandler {
 impl HttpHandler for MaintenanceHandler {
     async fn handle(
         &self,
-        method: http::Method,
         _: &str,
+        method: http::Method,
         params: &HashMap<String, String>,
     ) -> crate::Result<Response<String>> {
         match method {
             http::Method::GET => self.get_maintenance().await,
             http::Method::PUT => self.set_maintenance(params).await,
             _ => UnsupportedSnafu {
                 operation: format!("http method {method}"),
-            }.fail()
+            }
+            .fail(),
         }
     }
 }