refactor(cubesql): Fix manual_map warning

Author: mcheshkov

rust/cubesql/cubesql/Cargo.toml

@@ -105,7 +105,6 @@ len_without_is_empty = "allow"
 len_zero = "allow"
 let_and_return = "allow"
 manual_flatten = "allow"
-manual_map = "allow"
 manual_range_contains = "allow"
 manual_strip = "allow"
 map_clone = "allow"

rust/cubesql/cubesql/src/compile/engine/df/scan.rs

@@ -754,11 +754,7 @@ impl CubeScanOneShotStream {
     }
 
     fn poll_next(&mut self) -> Option<ArrowResult<RecordBatch>> {
-        if let Some(batch) = self.data.take() {
-            Some(Ok(batch))
-        } else {
-            None
-        }
+        self.data.take().map(Ok)
     }
 }
 

rust/cubesql/cubesql/src/compile/engine/udf/common.rs

@@ -2237,10 +2237,7 @@ pub fn create_pg_get_constraintdef_udf() -> ScalarUDF {
         let oids_arr = downcast_primitive_arg!(args[0], "oid", OidType);
         let result = oids_arr
             .iter()
-            .map(|oid| match oid {
-                Some(_oid) => Some("PRIMARY KEY (oid)".to_string()),
-                _ => None,
-            })
+            .map(|oid| oid.map(|_oid| "PRIMARY KEY (oid)".to_string()))
             .collect::<StringArray>();
 
         Ok(Arc::new(result))