-
Notifications
You must be signed in to change notification settings - Fork 58
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve event store indexes #66
Conversation
resolves #43
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some thoughts here: #43 (comment)
See also discussion here: #43 - I added some additional comments there. |
PostgresSimpleStreamStrategy is updated to have another indexing including |
@oqq that should do the trick @codeliner please review as well - I added a new interface - please check naming and namespace location. |
need to fix some tests, wait a minute .... |
ok ready @codeliner @oqq @basz |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like a dramatic improvement. Performance is stable at about ~90 commands per seconds (yielding ~3 events each)
First 180 seconds 92.5 commands/s
First 60 seconds 43.6 commands/s |
resolves #43
/cc @basz @oqq