You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opendal.apache.org by "zwpaper (via GitHub)" <gi...@apache.org> on 2023/03/21 03:52:22 UTC

[GitHub] [incubator-opendal] zwpaper opened a new issue, #1708: refactor or add a new function for Entry to return std::path::Path

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

   entry return `&str` currently:
   
   https://github.com/apache/incubator-opendal/blob/main/core/src/types/entry.rs#L58
   
   but it is really not convenient for path handling.
   
   maybe we can provide a new function to return the `std::path::Path` so that user does not have convert it manually everytime needed


-- 
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 #1708: refactor or add a new function for Entry to return std::path::Path

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

   Thanks for reaching out!
   
   This behaviour is by design since we can't and won't handle std::path::Path. The conversion is failable.
   
   Thus we will not add this function.


-- 
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 #1708: refactor or add a new function for Entry to return std::path::Path

Posted by "Xuanwo (via GitHub)" <gi...@apache.org>.
Xuanwo closed issue #1708: refactor or add a new function for Entry to return std::path::Path
URL: https://github.com/apache/incubator-opendal/issues/1708


-- 
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