Ji-Xinyou opened a new pull request, #3479:
URL: https://github.com/apache/incubator-opendal/pull/3479

   After implementing capability, the test framework for C is easier to 
implement. #3472 
   
   Two concerns along with this:
   - I think we need more documentions on the difference between full 
capabilities and native capabilities, the difference between them is not 
obvious to users. i.e. If they want to check the capabilities, should they use 
full/native?
   - Further work: we should replace all currently `opendal_byte` backed string 
like error message to leaked CString, which is more conventional for C. (This 
is my mistake at the very beginning 😢). I think this break the current API, so 
my plan would be changing this after our first release.


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