refactor(bindings/java): explict error handling (#3288)

Signed-off-by: tison <[email protected]>

bindings/java/src/error.rs

@@ -29,12 +29,6 @@ pub(crate) struct Error {
 }
 
 impl Error {
-    pub(crate) fn unexpected(err: impl Into<anyhow::Error> + Display) -> Error {
-        Error {
-            inner: opendal::Error::new(ErrorKind::Unexpected, &err.to_string()).set_source(err),
-        }
-    }
-
     pub(crate) fn throw(&self, env: &mut JNIEnv) {
         if let Err(err) = self.do_throw(env) {
             match err {
@@ -84,26 +78,14 @@ impl Error {
 }
 
 impl From<opendal::Error> for Error {
-    fn from(error: opendal::Error) -> Self {
-        Self { inner: error }
+    fn from(err: opendal::Error) -> Self {
+        Self { inner: err }
     }
 }
 
 impl From<jni::errors::Error> for Error {
-    fn from(error: jni::errors::Error) -> Self {
-        Error::unexpected(error)
-    }
-}
-
-impl From<std::str::Utf8Error> for Error {
-    fn from(error: std::str::Utf8Error) -> Self {
-        Error::unexpected(error)
-    }
-}
-
-impl From<std::string::FromUtf8Error> for Error {
-    fn from(error: std::string::FromUtf8Error) -> Self {
-        Error::unexpected(error)
+    fn from(err: jni::errors::Error) -> Self {
+        opendal::Error::new(ErrorKind::Unexpected, &err.to_string()).into()
     }
 }
 

bindings/java/src/lib.rs

@@ -135,7 +135,9 @@ fn make_presigned_request<'a>(env: &mut JNIEnv<'a>, req: PresignedRequest) -> Re
         let mut map = HashMap::new();
         for (k, v) in req.header().iter() {
             let key = k.to_string();
-            let value = v.to_str().map_err(Error::unexpected)?;
+            let value = v.to_str().map_err(|err| {
+                opendal::Error::new(opendal::ErrorKind::Unexpected, &err.to_string())
+            })?;
             map.insert(key, value.to_owned());
         }
         map