search

mookid8000 / Cirqus

:x: d60 event sourcing + CQRS framework
MIT License
233 stars 39 forks source link

Make sure events are returned in the correct sequence order. #75

Closed enriquein closed 8 years ago

enriquein commented 8 years ago

This prevents Expected an event with sequence number N. exceptions. I used to get these exceptions in about 3-5% of the emitted commands. This seems to fix it.

Here's a more detailed explanation of my issue: http://stackoverflow.com/questions/36487655/cirqus-expected-an-event-with-sequence-number-0-exception

mookid8000 commented 8 years ago

oh shit that was a good catch!! thanks, man! 👍 ⚡

mookid8000 commented 8 years ago

I'll see to it that your fix gets released immediately

enriquein commented 8 years ago

Much appreciated :smile:

mookid8000 commented 8 years ago

yeah - that must have bothered you 😨

mookid8000 commented 8 years ago

it's out in 0.67.1 as soon as the build server finishes running tests and packing

mookid8000 commented 8 years ago

it's out now 😁 thanks 👍

enriquein commented 8 years ago

That was super fast. Thanks!