This is an automated email from the ASF dual-hosted git repository.
xuanwo pushed a commit to branch add-lister-query
in repository https://gitbox.apache.org/repos/asf/incubator-opendal.git
The following commit(s) were added to refs/heads/add-lister-query by this push:
new 86c286585 Fix
86c286585 is described below
commit 86c28658579d92224112cdd0d4004b2f532b184f
Author: Xuanwo <[email protected]>
AuthorDate: Fri Aug 4 20:53:52 2023 +0800
Fix
Signed-off-by: Xuanwo <[email protected]>
---
core/src/docs/rfcs/2779_list_with_metakey.md | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/core/src/docs/rfcs/2779_list_with_metakey.md
b/core/src/docs/rfcs/2779_list_with_metakey.md
index 2e182e46c..43d94ef10 100644
--- a/core/src/docs/rfcs/2779_list_with_metakey.md
+++ b/core/src/docs/rfcs/2779_list_with_metakey.md
@@ -70,6 +70,11 @@ By introducing this change:
- Users don't need to capture `Operator` in the closure.
- Users don't need to do async call like `metadata()` again.
+If we don't have this change:
+
+- every place that could receive a `fn()` must use `Fn()` instead which
enforce users to have a generic parameter in their code.
+- It's harder for other languages binding to implement `op.metadata()` right.
+
# Guide-level explanation
The new API will be:
