esheppa commented on code in PR #9207:
URL: https://github.com/apache/arrow-datafusion/pull/9207#discussion_r1489247669
##########
datafusion/core/src/datasource/listing/url.rs:
##########
@@ -234,7 +239,7 @@ impl ListingTableUrl {
}
/// List all files identified by this [`ListingTableUrl`] for the provided
`file_extension`
- pub(crate) async fn list_all_files<'a>(
+ pub async fn list_all_files<'a>(
Review Comment:
I'm happy to separate this out if preferred. I made this change because it
allows easier debugging for future users who are unsure which files the
`ListingTableUrl` will actually list. This means one can see whether the issue
is with the url itself, or another issue
--
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]
