-
Notifications
You must be signed in to change notification settings - Fork 127
Conversation
…ems to hang when executed in a function, need to investigate.
This uses the unstable build of Npgsql 3 - might have to do some fiddling around with Nuget .config to get it looking at the unstable feed source here: https://www.myget.org/F/npgsql-unstable |
Conflicts: src/Cedar.EventStore.Tests/EventStoreAcceptanceTests.DeleteStream.cs Changed When_delete_stream_with_a_non_matching_expected_version_then_should_throw to include a couple of events in the stream, otherwise it tests deleting a non-existing stream.
@danbarua thanks for this! I've not forgotten it; just haven't had time lately for any oss stuff :( I need CES, and I need Postgres support, so greatly appreciated :) |
I’m putting it into production in a month, kittens might die… or there will be a flurry of PRs while I work the issues out. Dan Barua On Wednesday, 15 July 2015 at 22:29, Damian Hickey wrote:
|
…ion string name (can't leak Npgsql types due to assembly versioning) Change connection handling to create connection on-demand (issues with TPL + concurrency it seems) - lean on Npgsql connection pooling instead
…pected_version to get current version. If current_version is null then it's < expected version, if > expected_version then we don't have as many records to sort in order to find MAX(current_version) If using ExpectedVersion.Any then it will have to sort through all of the events.
… limit. ExpectedVersion.Any should now perform as well as a given expected version.
…y collection of events but will eventually return the end of the stream, with a checkpoint.
Don't merge this just yet, some more wrinkles to iron out re: concurrency under heavy load. |
…t has already been completed
With that last commit, I'm going to monitor our error queues for a few days and fingers crossed we might be production-ready. :) |
… stream metadata before appending
* Upgrade nuget packages * Use correct version tavis Co-authored-by: Arne Schoonvliet <arne@intigriti.com>
^ what that says