chore: Remove redundant variable assign (#2127)

* chore: Remove redundant variable assign * fix other clippy warnings
getdozer · Oct 6, 2023 · d53a131 · d53a131
1 parent e01363b
commit d53a131
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 4 deletions.
diff --git a/dozer-api/src/grpc/client_server.rs b/dozer-api/src/grpc/client_server.rs
@@ -143,7 +143,6 @@ impl ApiServer {
         } else {
             unauthenticated_reflection_service = Some(reflection_service);
         };
-        let health_service = health_service;
 
         let mut auth_service = None;
         let security = get_api_security(self.security.to_owned());

diff --git a/dozer-ingestion/src/connectors/object_store/connector.rs b/dozer-ingestion/src/connectors/object_store/connector.rs
@@ -210,7 +210,6 @@ impl<T: DozerObjectStore> Connector for ObjectStoreConnector<T> {
                                 joinset.spawn(async move {
                                     let mut csv_table = CsvTable::new(csv_config, config);
                                     csv_table.update_state = state;
-                                    let table_info = table_info;
                                     csv_table.watch(table_index, &table_info, sender).await?;
                                     Ok::<_, ConnectorError>(())
                                 });

diff --git a/dozer-ingestion/src/connectors/snowflake/connection/client.rs b/dozer-ingestion/src/connectors/snowflake/connection/client.rs
@@ -351,13 +351,13 @@ impl<'env> Client<'env> {
             .map(|(name, (_, schema))| match schema {
                 Ok(mut schema) => {
                     let mut indexes = vec![];
-                    keys.get(&name).map_or((), |columns| {
+                    if let Some(columns) = keys.get(&name) {
                         schema.fields.iter().enumerate().for_each(|(idx, f)| {
                             if columns.contains(&f.name) {
                                 indexes.push(idx);
                             }
                         });
-                    });
+                    }
 
                     let cdc_type = if indexes.is_empty() {
                         CdcType::Nothing