comphead commented on code in PR #16487:
URL: https://github.com/apache/datafusion/pull/16487#discussion_r2164165235


##########
datafusion/core/src/datasource/listing_table_factory.rs:
##########
@@ -125,6 +125,13 @@ impl TableProviderFactory for ListingTableFactory {
             // specifically for parquet file format.
             // See: https://github.com/apache/datafusion/issues/7317
             None => {
+                // if the folder then rewrite a file path as 'path/*.parquet'

Review Comment:
   it should be ok, compressed files are usually `*.codec.parquet` and more 
broad wildcard `*.parquet` should read them. My local test I did against 
`part-00000-9b95f137-d11f-44b6-84b7-d49c95bc7c5b-c000.snappy.parquet`



-- 
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...@datafusion.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to