AdamGS commented on code in PR #14873:
URL: https://github.com/apache/datafusion/pull/14873#discussion_r1970448939


##########
datafusion/core/src/datasource/file_format/parquet.rs:
##########
@@ -471,6 +468,153 @@ impl FileFormat for ParquetFormat {
     }
 }
 
+/// Coerces the file schema if the table schema uses a view type.
+#[cfg(not(target_arch = "wasm32"))]

Review Comment:
   Already ran into one small issue with csv/json when I tried to move the 
`Decoder` trait, but I think that once I make that big split it'll be easier to 
find a home for all the shared things, hopefully without having to change them 
too much.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to