make clippy check

cnosdb · Aug 2, 2023 · f4a2f42 · f4a2f42
1 parent e845f36
commit f4a2f42
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 10 deletions.
diff --git a/query_server/query/src/metadata/mod.rs b/query_server/query/src/metadata/mod.rs
@@ -58,7 +58,7 @@ pub trait ContextProviderExtension: ContextProvider {
         name: TableReference,
     ) -> datafusion::common::Result<Arc<TableSourceAdapter>>;
     fn database_table_exist(&self, _database: &str, _table: Option<&ResolvedTable>) -> Result<(), MetaError> {
-        return Ok(());
+        Ok(())
     }
 }
 
@@ -245,16 +245,13 @@ impl ContextProviderExtension for MetadataProvider {
                 database: database.to_string() 
         })?;
 
-        match table {
-            Some(table) => {
-                data_info.tables.get( table.table()).ok_or(MetaError::TableNotFound {
-                    table: table.to_string(), 
-                })?;
-            }
-            None => (),
+        if let Some(table) = table {
+            data_info.tables.get( table.table()).ok_or(MetaError::TableNotFound {
+                table: table.to_string(), 
+            })?;
         }
 
-        return Ok(());
+        Ok(())
     }
 }
 

diff --git a/query_server/query/src/sql/planner.rs b/query_server/query/src/sql/planner.rs
@@ -679,7 +679,7 @@ impl<'a, S: ContextProviderExtension + Send + Sync + 'a> SqlPlanner<'a, S> {
             .project(projections)?
             .build()?;
 
-        let plan = Plan::Query(QueryPlan { df_plan: df_plan });
+        let plan = Plan::Query(QueryPlan { df_plan });
 
         // privileges
         let tenant_id = *session.tenant_id();