Xuanwo commented on PR #6764:
URL: https://github.com/apache/opendal/pull/6764#issuecomment-3506479797

   > Perhaps we should consider removing the `Scheme` enum from the codebase 
later? After merging this PR, the `Scheme` enum will only be used for the two 
public APIs:
   > 
   > * `Operator::via_iter`: I think this API is more suitable for test cases 
than for production environments, and the existing code in use reflects this 
view.
   > * `Operator::via_map`: had been deprecated, use `via_iter` instead. BTW, 
do we have a strategy for removing deprecated APIs? I haven't found any similar 
discussions yet.
   
   Yep, that's our goal to finally remove it. We can keep `Scheme` as an alias 
to `&'static str` to avoid breaking APIs to much.


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