handleMessageTimeout implemented in executeBatchHandler
#568
Ashwin2397
started this conversation in
Feature Request
Replies: 1 comment 1 reply
-
|
Hey, thanks for raising this. This is interesting, something we could take a look at for sure. |
Beta Was this translation helpful? Give feedback.
1 reply
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.
-
Background
The
executeBatchHandler(src/consumer.ts#58) currently does not account for timeouts, unlikeexecuteHandler. This implies that ifhandleMessageBatchis used, it can not catch timeout errors in the context of the dependency. It'd have to be implemented "around" the dependency (ie. by the client).Ideally this implementation stays within the context of the dependency. It'd also promote symmetry with the existing
executeHandler.Objectives
handleMessageBatchBeta Was this translation helpful? Give feedback.
All reactions