apache / opendal

Apache OpenDAL: One Layer, All Storage.

https://opendal.apache.org

Apache License 2.0

3.47k stars 486 forks source link

RFC-3911: Remove recursive from deleter #5288

Closed Xuanwo closed 2 weeks ago

Xuanwo commented 2 weeks ago

Which issue does this PR close?

Part of #3911

Rationale for this change

While revisit this idea, I found that there is no need for deleter to have recursive. Users who want remove dir recursively can just use op.lister_with(path).recursive(true) and pass to deleter.

What changes are included in this PR?

Remove recursive from deleter

Are there any user-facing changes?

RFC Changes.

Xuanwo commented 2 weeks ago

Thank you @dqhl76 for helping review this!