BlakeOrth commented on code in PR #18146:
URL: https://github.com/apache/datafusion/pull/18146#discussion_r2491888098
##########
datafusion/core/tests/datasource/object_store_access.rs:
##########
@@ -174,10 +165,8 @@ async fn query_partitioned_csv_file() {
+---------+-------+-------+---+----+-----+
------- Object Store Request Summary -------
RequestCountingObjectStore()
- Total Requests: 4
- - LIST (with delimiter) prefix=data/a=2
- - LIST (with delimiter) prefix=data/a=2/b=20
- - LIST (with delimiter) prefix=data/a=2/b=20/c=200
+ Total Requests: 2
+ - LIST prefix=data
Review Comment:
Yes absolutely, with some additional work! I had initially planned on doing
this out of the gate but chose to omit that from this first PR to keep the
review/change scope lower. Doing so with the work in this PR requires some
additional methods to be implemented in the low level object store interactions
in DataFusion, but they aren't terribly complex based what I've already learned.
--
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]
