You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opendal.apache.org by "Xuanwo (via GitHub)" <gi...@apache.org> on 2023/08/02 16:20:36 UTC

[GitHub] [incubator-opendal] Xuanwo opened a new issue, #2761: idea: Add `op.lister()` and make `op.list()` return `Vec`

Xuanwo opened a new issue, #2761:
URL: https://github.com/apache/incubator-opendal/issues/2761

   The behavior between `op.read()` and `op.list()` is not consist.
   
   `op.read()` returns the content in an easily usable format. However, `op.list()` returns a `Lister` object which requires users to work with the stream according to their specific needs, even when they simply want a list of files.
   
   So I think it's a good idea to:
   
   - Rename `op.list()` to `op.lister()`
   - Add `op.list() -> Resutl<Vec<Entry>>` which will read a whole dir's content.
   - So does for `op.scan() -> Result<Vec<Entry>>`


-- 
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: commits-unsubscribe@opendal.apache.org.apache.org

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


[GitHub] [incubator-opendal] Xuanwo commented on issue #2761: idea: Add `op.lister()` and make `op.list()` return `Vec`

Posted by "Xuanwo (via GitHub)" <gi...@apache.org>.
Xuanwo commented on issue #2761:
URL: https://github.com/apache/incubator-opendal/issues/2761#issuecomment-1664947307

   Started as a RFC in https://github.com/apache/incubator-opendal/pull/2774


-- 
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: commits-unsubscribe@opendal.apache.org

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


[GitHub] [incubator-opendal] Xuanwo closed issue #2761: idea: Add `op.lister()` and make `op.list()` return `Vec`

Posted by "Xuanwo (via GitHub)" <gi...@apache.org>.
Xuanwo closed issue #2761: idea: Add `op.lister()` and make `op.list()` return `Vec<Entry>`
URL: https://github.com/apache/incubator-opendal/issues/2761


-- 
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: commits-unsubscribe@opendal.apache.org

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