You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opendal.apache.org by "WenyXu (via GitHub)" <gi...@apache.org> on 2023/12/11 06:19:30 UTC
[PR] feat: add `with_default` to `OperatorBuilder` [incubator-opendal]
WenyXu opened a new pull request, #3738:
URL: https://github.com/apache/incubator-opendal/pull/3738
Added `with_default` to `OperatorBuilder`. It allows users to add layers before the default layers.
close #3737
--
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: [PR] feat: add `with_default` to `OperatorBuilder` [incubator-opendal]
Posted by "WenyXu (via GitHub)" <gi...@apache.org>.
WenyXu closed pull request #3738: feat: add `with_default` to `OperatorBuilder`
URL: https://github.com/apache/incubator-opendal/pull/3738
--
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: [PR] feat: add `with_default` to `OperatorBuilder` [incubator-opendal]
Posted by "WenyXu (via GitHub)" <gi...@apache.org>.
WenyXu commented on PR #3738:
URL: https://github.com/apache/incubator-opendal/pull/3738#issuecomment-1849480420
> #3737
Thanks, it works for me 🥺, I'm going to close the PR.
--
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: [PR] feat: add `with_default` to `OperatorBuilder` [incubator-opendal]
Posted by "Xuanwo (via GitHub)" <gi...@apache.org>.
Xuanwo commented on PR #3738:
URL: https://github.com/apache/incubator-opendal/pull/3738#issuecomment-1849448187
I don't want to expose API like `with_default` to users. If you have strong request for this feature, you can wrap the `Accessor` with layers you want outsides and calling `Operator::new()`.
--
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: [PR] feat: add `with_default` to `OperatorBuilder` [incubator-opendal]
Posted by "WenyXu (via GitHub)" <gi...@apache.org>.
WenyXu commented on PR #3738:
URL: https://github.com/apache/incubator-opendal/pull/3738#issuecomment-1849412857
Interesting, some tests seem unstable. https://github.com/apache/incubator-opendal/actions/runs/7163688905
--
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