Skip to content
This repository was archived by the owner on Aug 15, 2024. It is now read-only.

Add support for PostgreSQL #2

Merged
merged 37 commits into from
Nov 17, 2015
Merged

Add support for PostgreSQL #2

merged 37 commits into from
Nov 17, 2015

Conversation

danbarua
Copy link
Contributor

^ what that says

@danbarua
Copy link
Contributor Author

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

danbarua added 8 commits June 22, 2015 09:25
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.
@damianh
Copy link
Member

damianh commented Jul 15, 2015

@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 :)

@danbarua
Copy link
Contributor Author

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:

@danbarua (https://github.com/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 :)


Reply to this email directly or view it on GitHub (#2 (comment)).

danbarua added 13 commits July 16, 2015 11:15
…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.
@danbarua
Copy link
Contributor Author

Don't merge this just yet, some more wrinkles to iron out re: concurrency under heavy load.

@danbarua
Copy link
Contributor Author

danbarua commented Oct 8, 2015

With that last commit, I'm going to monitor our error queues for a few days and fingers crossed we might be production-ready. :)

damianh added a commit that referenced this pull request Nov 17, 2015
Add support for PostgreSQL
@damianh damianh merged commit 5979614 into SQLStreamStore:master Nov 17, 2015
damianh added a commit that referenced this pull request Oct 28, 2018
ArneSchoonvliet added a commit to ErazerBrecht/SQLStreamStore that referenced this pull request Apr 20, 2022
* Upgrade nuget packages

* Use correct version tavis

Co-authored-by: Arne Schoonvliet <arne@intigriti.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants