vovacf201 opened a new pull request, #2134:
URL: https://github.com/apache/iceberg-rust/pull/2134
## Which issue does this PR close?
- Closes #120.
## What changes are included in this PR?
Eagerly collect manifest list paths into Vec<String> before the
await point in clean_files to avoid a higher-ranked trait bound
(HRTB) compiler error.
The closure `|s| s.to_string()` on HashSet<&'a str> gets inferred
with a concrete lifetime that the compiler cannot generalize across
async generator boundaries, producing:
"closure with signature fn(&'0 str) -> String must implement
FnOnce<(&'1 str,)>, for any two lifetimes '0 and '1"
Collecting into Vec<String> before the await removes the
lifetime-bound closure from the future's state machine entirely.
Adds a variant of cleanup_expired_files that accepts an explicit FileIO,
for cases where the committed table's file_io lacks S3 configuration
(e.g. region) that was present on the original table.
--
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]
