thinkharderdev commented on a change in pull request #1622:
URL: https://github.com/apache/arrow-datafusion/pull/1622#discussion_r789970485



##########
File path: datafusion/src/datasource/file_format/parquet.rs
##########
@@ -367,13 +366,28 @@ mod tests {
     };
 
     use super::*;
+    use crate::datasource::listing::ListingOptions;
     use crate::execution::runtime_env::{RuntimeConfig, RuntimeEnv};
     use arrow::array::{
         BinaryArray, BooleanArray, Float32Array, Float64Array, Int32Array,
         TimestampNanosecondArray,
     };
     use futures::StreamExt;
 
+    #[tokio::test]
+    async fn test_merge_schema() -> Result<()> {

Review comment:
       @alamb Not sure we need a test case here since it gets tested implicitly 
in the `ParquetExec` tests? If you'd rather have an explicit test case here, 
then I can use the same utility from those test cases over here. 




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to