dheeraj12347 commented on code in PR #7176:
URL: https://github.com/apache/opendal/pull/7176#discussion_r2757497710
##########
core/core/src/raw/ops.rs:
##########
@@ -133,6 +134,8 @@ pub struct OpList {
///
/// Default to `false`
deleted: bool,
+ /// The timeout for this operation.
+ timeout: Option<Duration>,
Review Comment:
> Thank you, but `Layer`'s behavior should never be injected into ops level.
I have pushed the refactored TimeoutLayer. It now handles timeouts entirely
within the layer, and I've reverted all changes to the core ops.rs. The code is
now architecturally clean and passes local checks. Looking forward to your
guidance on the best way to re-introduce per-operation overrides!
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
