search

wework / grabbit

A lightweight transactional message bus on top of RabbitMQ
Apache License 2.0
98 stars 20 forks source link

added custom saga correlation capabilties #211

Closed rhinof closed 4 years ago

rhinof commented 4 years ago

210

coveralls commented 4 years ago

Coverage Status

Coverage increased (+0.1%) to 76.335% when pulling 19fe1118a5517b465d64549abc97ac50e23b1d84 on custom_saga_locator into e7e27a51cf24765fad39746a116317267461a540 on v1.x.

rhinof commented 4 years ago

@vladshub I have updated the solution along the lines of what we discussed

rhinof commented 4 years ago

@vladshub there is a severe defect in this design in which there might be a race condition that is not trivial to solve when there is more than one consumer. I am closing the PR and abandoning this approach and remaining with the current solution of TargetSaga()