Skip to content
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

Add Equinox.EventStoreDb - gRPC support #196

Merged
merged 29 commits into from
May 12, 2022
Merged

Add Equinox.EventStoreDb - gRPC support #196

merged 29 commits into from
May 12, 2022

Conversation

bartelink
Copy link
Collaborator

@bartelink bartelink commented Feb 17, 2020

This introduces a port of the Equinox.EventStore module entitled Equinox.EventStore.Grpc which:

  • uses EventStore.Client.Grpc.Streams v 22.0.0

In doing this, the Equinox.EventStore package remains but:

  • the Sample's integration tests shift to use the gRPC package instead
  • is intended for use against server versions 21.10 and later, i.e. there is no logic to use V5 server compatibility mode etc
  • references EventStore.Client v 22.0.0-preview

As part of the above, the guidance changes from running a local ESDB (v5) instance obtained via Chocolatey (on Windows) or brew (on MacOS) to using a 3 node cluster running under docker compose

related: #188

@bartelink
Copy link
Collaborator Author

This module should be renamed to Equinox.EventStoreDb (to disambiguate from Equinox.EventStore), which will continue to depend on EventStore.Client, (and align with EventStore's new consistent naming).

@bartelink bartelink added help wanted up-for-grabs https://up-for-grabs.net labels Feb 15, 2021
@bartelink
Copy link
Collaborator Author

Would love to see this implemented as it's abundantly clear that the gRPC interface is the only game in town going forward for EventStoreDB and simply makes sense to migrate to.

I don't have time to do this at present (and my current team I'm on are still on V5).

@bartelink bartelink added this to the 3.0 milestone Feb 17, 2021
@dharmaturtle
Copy link
Contributor

dharmaturtle commented Apr 25, 2021

Hi! What would it take to complete this PR? When I diff EventSToreGrpc.fs with the main EventStore.fs, it isn't clear to me what's missing (other than the one TODO). Is it just updating the client to the LTS (20.10) and resolving the conflicts?

Edit: Ah - it doesn't compile.

@bartelink bartelink merged commit c0e01ea into master May 12, 2022
@bartelink bartelink deleted the es6 branch May 12, 2022 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants