-
Notifications
You must be signed in to change notification settings - Fork 6
add bulk pubsub example for js #26
base: main
Are you sure you want to change the base?
Conversation
@kendallroden fyi, wdyt? |
tips: the bulk pubsub would use routes |
lets change the events to match the order domain please |
As per discussion the HTTP request sample can be changed flexibly. To make it more align with existing samples, something maybe:
|
what do you mean "the order domain" ? |
chatted with @kendallroden and she expected the HTTP sample text can align with the existing order sample for pubsub, so I left this comment guess we can design it more carefully when updating related docs |
Resolves: https://github.com/diagridio/issues/issues/2733
As discussed in https://github.com/diagridio/issues/issues/2733#issuecomment-1973571828, we need to add bulk pubsub example code additionally to make it compatible with dapr.
tips: the bulk pubsub would use routes
pubsub/bulk-orders
andpubsub/bulk-neworders
, so make sure you setup catalyst resources correctly before trying it :)I did manual test, and believe this help to drop unexpected retries and let consumer only receive bulk events exactly once: (feel free to change
entryId
andevent
to any string)