This repository has been archived by the owner on Jul 27, 2019. It is now read-only.

Repository should use timestamp instead of id for query ordering #14

Open

renou opened this issue Feb 10, 2017 · 0 comments

Contributor

renou commented Feb 10, 2017

should be:
https://github.com/johnbywater/eventsourcing/blob/26c2b41a5c5bc1c754cdc809bb40e1dce175d144/eventsourcing/infrastructure/stored_event_repos/with_sqlalchemy.py#L137
right now:
https://github.com/ApplauseOSS/djangoevents/blob/devel/djangoevents/repository.py#L76

the reason behind is if multiple events get persist to the database, the order of inserts may not be the order of event creation.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.