Xuanwo commented on code in PR #2112:
URL:
https://github.com/apache/incubator-opendal/pull/2112#discussion_r1176338231
##########
bindings/java/src/main/java/org/apache/opendal/Operator.java:
##########
@@ -50,17 +49,24 @@ public Operator(String schema, Map<String, String> params) {
private native void write(long ptr, String fileName, String content);
+ private native void asyncWrite(long ptr, String fileName, String content,
CompletableFuture<Boolean> future);
+
private native String read(long ptr, String fileName);
private native void delete(long ptr, String fileName);
private native long stat(long ptr, String file);
-
public void write(String fileName, String content) {
write(this.ptr, fileName, content);
}
+ public CompletableFuture<Boolean> asyncWrite(String fileName, String
content) {
Review Comment:
How will java community design API for both blocking and async operations?
Do they prefer to have `asyncWrite` to a seperate class called `AsyncOperator`?
--
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]
