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

Stop reading the order stream once the checkpoint has passed. #1465

Merged
merged 1 commit into from Oct 13, 2017

Conversation

2 participants
@lscpike
Contributor

lscpike commented Oct 12, 2017

Looking at the Windows file caching I noticed one of our event store clusters was always loading the entire database into the file cache. This turned out to be a multi-stream projection causing the projection system to read an entire stream all the way backwards during initialization. In particular the "order" stream. This takes about 2 hours and blows out the file cache...

It looks like the break was attempting to break out the switch but was actually breaking out the foreach.

Test included.

@hayley-jean

This comment has been minimized.

Show comment
Hide comment
@hayley-jean

hayley-jean Oct 13, 2017

Contributor

Thanks! 👍

Contributor

hayley-jean commented Oct 13, 2017

Thanks! 👍

@hayley-jean hayley-jean merged commit cb5385b into EventStore:release-v4.0.4 Oct 13, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
wercker/build-mono4 Wercker pipeline passed
Details

@lscpike lscpike deleted the lscpike:bugfix/multistream-projection-full-history-read branch Apr 30, 2018

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