[I] Support extend layers for Java binding [incubator-opendal]

["tisonkun (via GitHub)" <gi...@apache.org>]

tisonkun opened a new issue, #3367:
URL: https://github.com/apache/incubator-opendal/issues/3367

   (no comment)


-- 
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: commits-unsubscribe@opendal.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] Support extend layers for Java binding [incubator-opendal]

["Xuanwo (via GitHub)" <gi...@apache.org>]

Xuanwo commented on issue #3367:
URL: https://github.com/apache/incubator-opendal/issues/3367#issuecomment-1780746773

   > The first challenge I meet is how to pipeline the accessor and adapt operator class into this absraction :D
   
   I strongly oppose exposing `Accessor` to the bindings realm. It's OpenDAL's raw API, which users should never interact with. Technically, it's challenging for Java to access generic types in Rust through FFI.


-- 
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: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] Support extend layers for Java binding [incubator-opendal]

["tisonkun (via GitHub)" <gi...@apache.org>]

tisonkun commented on issue #3367:
URL: https://github.com/apache/incubator-opendal/issues/3367#issuecomment-1776755697

   @Xuanwo the layer implemented in Java API should include a way to export Rust's layers, while it's not conflict with making a pure Java layer to leverage Java ecosystem. The first challenge I meet is how to pipeline the accessor and adapt operator class into this absraction :D


-- 
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: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] Support extend layers for Java binding [incubator-opendal]

["Xuanwo (via GitHub)" <gi...@apache.org>]

Xuanwo commented on issue #3367:
URL: https://github.com/apache/incubator-opendal/issues/3367#issuecomment-1776740369

   Recently, I've been considering the concept of layers in bindings. I previously believed that all bindings should have their own unique layer abstractions and implementations. However, I discovered that it's illogical for each binding to implement the same layer individually. This contradicts our OpenDAL's VISON.
   
   Looking back at the Java binding, perhaps the main issue lies in how we've designed the API to expose Rust layers in Java. We could consider exposing them as `org.apache.opendal.layers.RetryLayer`, and construct a similar API like `op.layer(new RetryLayer())`.


-- 
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: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] Support extend layers for Java binding [incubator-opendal]

["Xuanwo (via GitHub)" <gi...@apache.org>]

Xuanwo closed issue #3367: Support extend layers for Java binding
URL: https://github.com/apache/incubator-opendal/issues/3367


-- 
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: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org