Xuanwo commented on issue #3127:
URL: 
https://github.com/apache/incubator-opendal/issues/3127#issuecomment-1724780399

   The main block here is `Entry` is a composition of `Path` and `Metadata`. 
   
   There are two downsides:
   
   First, returning the `Entry` for `stat` requires an additional clone of the 
path (although it is not a significant issue in most cases).
   
   Secondly, the return of `Entry` causes all calls to our `stat` function to 
be changed from:
   
   ```diff
   - let v = op.stat(path).await?.content_length();
   + let v = op.stat(path).await?.metadata().content_length();
   ```
   
   I'm not sure if it's a good idea.


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

Reply via email to