From a663eafe9c1c1ffeef4077f6f7d1002654e7666e Mon Sep 17 00:00:00 2001
From: Chojan Shang <psiace@apache.org>
Date: Sat, 22 Jun 2024 14:11:34 +0800
Subject: [PATCH] feat: make AwaitTreeLayer covers oio::Read and oio::Write

Signed-off-by: Chojan Shang <psiace@apache.org>
---
 core/src/layers/await_tree.rs | 99 +++++++++++++++++++++++++++++++----
 1 file changed, 90 insertions(+), 9 deletions(-)

diff --git a/core/src/layers/await_tree.rs b/core/src/layers/await_tree.rs
index 49834d6dc30b..7bb20d42fb49 100644
--- a/core/src/layers/await_tree.rs
+++ b/core/src/layers/await_tree.rs
@@ -16,6 +16,8 @@
 // under the License.
 
 use await_tree::InstrumentAwait;
+use futures::{Future, FutureExt};
+use oio::{ListOperation, ReadOperation, WriteOperation};
 
 use crate::raw::*;
 use crate::*;
@@ -67,12 +69,12 @@ pub struct AwaitTreeAccessor<A: Access> {
 
 impl<A: Access> LayeredAccess for AwaitTreeAccessor<A> {
     type Inner = A;
-    type Reader = A::Reader;
-    type BlockingReader = A::BlockingReader;
-    type Writer = A::Writer;
-    type BlockingWriter = A::BlockingWriter;
-    type Lister = A::Lister;
-    type BlockingLister = A::BlockingLister;
+    type Reader = AwaitTreeWrapper<A::Reader>;
+    type BlockingReader = AwaitTreeWrapper<A::BlockingReader>;
+    type Writer = AwaitTreeWrapper<A::Writer>;
+    type BlockingWriter = AwaitTreeWrapper<A::BlockingWriter>;
+    type Lister = AwaitTreeWrapper<A::Lister>;
+    type BlockingLister = AwaitTreeWrapper<A::BlockingLister>;
 
     fn inner(&self) -> &Self::Inner {
         &self.inner
@@ -82,6 +84,7 @@ impl<A: Access> LayeredAccess for AwaitTreeAccessor<A> {
         self.inner
             .read(path, args)
             .instrument_await(format!("opendal::{}", Operation::Read))
+            .map(|v| v.map(|(rp, r)| (rp, AwaitTreeWrapper::new(r))))
             .await
     }
 
@@ -89,6 +92,7 @@ impl<A: Access> LayeredAccess for AwaitTreeAccessor<A> {
         self.inner
             .write(path, args)
             .instrument_await(format!("opendal::{}", Operation::Write))
+            .map(|v| v.map(|(rp, r)| (rp, AwaitTreeWrapper::new(r))))
             .await
     }
 
@@ -124,6 +128,7 @@ impl<A: Access> LayeredAccess for AwaitTreeAccessor<A> {
         self.inner
             .list(path, args)
             .instrument_await(format!("opendal::{}", Operation::List))
+            .map(|v| v.map(|(rp, r)| (rp, AwaitTreeWrapper::new(r))))
             .await
     }
 
@@ -142,14 +147,90 @@ impl<A: Access> LayeredAccess for AwaitTreeAccessor<A> {
     }
 
     fn blocking_read(&self, path: &str, args: OpRead) -> Result<(RpRead, Self::BlockingReader)> {
-        self.inner.blocking_read(path, args)
+        self.inner
+            .blocking_read(path, args)
+            .map(|(rp, r)| (rp, AwaitTreeWrapper::new(r)))
     }
 
     fn blocking_write(&self, path: &str, args: OpWrite) -> Result<(RpWrite, Self::BlockingWriter)> {
-        self.inner.blocking_write(path, args)
+        self.inner
+            .blocking_write(path, args)
+            .map(|(rp, r)| (rp, AwaitTreeWrapper::new(r)))
     }
 
     fn blocking_list(&self, path: &str, args: OpList) -> Result<(RpList, Self::BlockingLister)> {
-        self.inner.blocking_list(path, args)
+        self.inner
+            .blocking_list(path, args)
+            .map(|(rp, r)| (rp, AwaitTreeWrapper::new(r)))
+    }
+}
+
+pub struct AwaitTreeWrapper<R> {
+    inner: R,
+}
+
+impl<R> AwaitTreeWrapper<R> {
+    fn new(inner: R) -> Self {
+        Self { inner }
+    }
+}
+
+impl<R: oio::Read> oio::Read for AwaitTreeWrapper<R> {
+    async fn read(&mut self) -> Result<Buffer> {
+        self.inner
+            .read()
+            .instrument_await(format!("opendal::{}", ReadOperation::Read))
+            .await
+    }
+}
+
+impl<R: oio::BlockingRead> oio::BlockingRead for AwaitTreeWrapper<R> {
+    fn read(&mut self) -> Result<Buffer> {
+        self.inner.read()
+    }
+}
+
+impl<R: oio::Write> oio::Write for AwaitTreeWrapper<R> {
+    fn write(&mut self, bs: Buffer) -> impl Future<Output = Result<usize>> + MaybeSend {
+        self.inner
+            .write(bs)
+            .instrument_await(format!("opendal::{}", WriteOperation::Write.into_static()))
+    }
+
+    fn abort(&mut self) -> impl Future<Output = Result<()>> + MaybeSend {
+        self.inner
+            .abort()
+            .instrument_await(format!("opendal::{}", WriteOperation::Abort.into_static()))
+    }
+
+    fn close(&mut self) -> impl Future<Output = Result<()>> + MaybeSend {
+        self.inner
+            .close()
+            .instrument_await(format!("opendal::{}", WriteOperation::Close.into_static()))
+    }
+}
+
+impl<R: oio::BlockingWrite> oio::BlockingWrite for AwaitTreeWrapper<R> {
+    fn write(&mut self, bs: Buffer) -> Result<usize> {
+        self.inner.write(bs)
+    }
+
+    fn close(&mut self) -> Result<()> {
+        self.inner.close()
+    }
+}
+
+impl<R: oio::List> oio::List for AwaitTreeWrapper<R> {
+    async fn next(&mut self) -> Result<Option<oio::Entry>> {
+        self.inner
+            .next()
+            .instrument_await(format!("opendal::{}", ListOperation::Next))
+            .await
+    }
+}
+
+impl<R: oio::BlockingList> oio::BlockingList for AwaitTreeWrapper<R> {
+    fn next(&mut self) -> Result<Option<oio::Entry>> {
+        self.inner.next()
     }
 }