Replies: 3 comments 1 reply
-
I'm talking to some known
|
Beta Was this translation helpful? Give feedback.
1 reply
-
+1, very good idea. |
Beta Was this translation helpful? Give feedback.
0 replies
-
Thank you all for the discusson, I have start an RFC for this change. Welcome to review it! |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
OpenDAL supports both blocking and asynchronous APIs. However, over time, this design has created a significant maintenance burden. Few services truly have blocking API support, yet we all have to invest considerable effort in duplicating code throughout the project.
Maybe it's time for us to remove blocking API entirely from
raw::Access
. And only keepBlockingOperator
as a wrapper ofOperator
.After this change:
raw::Access
will only have async API which can reduce a lot of duplicated code, especially in layers.BlockingOperator
won't be affected.BlockingLayer
could be removed and related APIs been moved toBlockingOperator
instead.Benefits:
Beta Was this translation helpful? Give feedback.
All reactions