alamb commented on code in PR #6619:
URL: https://github.com/apache/arrow-rs/pull/6619#discussion_r1834883065
##########
object_store/src/client/list.rs:
##########
@@ -44,37 +44,38 @@ pub(crate) trait ListClientExt {
prefix: Option<&Path>,
delimiter: bool,
offset: Option<&Path>,
- ) -> BoxStream<'_, Result<ListResult>>;
+ ) -> BoxStream<'static, Result<ListResult>>;
Review Comment:
Right -- I guess I am wondering does this effectively mean that the returned
streams can't be borrowed (aka they effectively have to be Arc'd or something
like that where the stream is owned 🤔 )
Basically I am trying to think about what usecases, if any, that this
additional constrait would be break (I realize some people may need to update
the type signatures, I am thinking of more fundamental changes)
One thing maybe we could do is try to update DataFusion (and our internal
INfluxDB code) with this change and see what, if anything, breaks for us 🤔
--
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]
