You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@opendal.apache.org by "Zheaoli (via GitHub)" <gi...@apache.org> on 2023/12/18 16:11:30 UTC

Re: [PR] RFC-3734: Buffered reader [incubator-opendal]

Zheaoli commented on PR #3734:
URL: https://github.com/apache/incubator-opendal/pull/3734#issuecomment-1860927069

   For me, mostly LGTM. BTW, I have a suggestion here, maybe we need to add a `Capability` to let each service's developer decide support the buffer read or  not and check the maximum size their own.
   
   Because some of the service like PGSQL,MySQL etc.. are dedicated to the small file.
   
   This is probably an implementation issue, I'm not sure we need add extra description in RFC docs


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