-
Type:
Task
-
Resolution: Unresolved
-
Priority:
Unknown
-
None
-
Component/s: Backpressure
-
None
-
Needed
Summary
The spec changes introduced in DRIVERS-3391 added Note 1 to the withTransaction Sequence of Actions, describing how timeout errors should be reported. During the review of PR #1899, several areas were identified where the wording could be clearer.
Clarifications Needed
- Only wrap retriable errors with makeTimeoutError
- In sections 7.4 & 10.3 of the sequence-of-actions the errors are not retriable, and they are mainly the reason for returning immediately from withTransaction)
- Update the prose test to assert that the error labels on the propagated timeout error match the error labels of the wrapped error.
- Consistent wording:
- Replace SHOULD expose error label(s) from the transient error with a {{SHOULD }} requirement to copy all error labels from the wrapped error to the wrapping timeout error.
- Replace "underlying error" with "wrapped error" for consistent terminology.
- Replace use of "raise" with "propagate" for consistency.
- Turn "Note 1" into a section and reference it.
- has to be started together with
-
DRIVERS-3391 Clarify expected error if backoff exceeds CSOT's deadline in withTransaction
-
- Implementing
-
- is related to
-
DRIVERS-3391 Clarify expected error if backoff exceeds CSOT's deadline in withTransaction
-
- Implementing
-