djangoevents icon indicating copy to clipboard operation
djangoevents copied to clipboard

Repository should use timestamp instead of id for query ordering

Open renou opened this issue 9 years ago • 0 comments

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.

renou avatar Feb 10 '17 18:02 renou