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 Load test; Optimize timeouts and connection settings using 4.1.1-rc1 client #17

Merged
merged 5 commits into from
Apr 24, 2018

Conversation

bartelink
Copy link
Collaborator

@bartelink bartelink commented Mar 23, 2018

In order to provide and validate reliability guarantees running against EventStore within our production setup, there's a need to validate Foldunk and the GesConnector together fulfil our needs wrt:

  • performance (sustained rate of read+write transaction mix over time)
  • memory consumption (no leaks over a sustained run)
  • reliability (connection should be maintained independent of the server connection being lost for any of the following reasons):
    • client-driven heartbeat timeout
    • server-driven heartbeat timeout
    • whacking the server process
    • yanking a TCP connection by triggering an involuntary close of the socket at OS level

Includes a load test driver authored by @eiriktsarpalis 🙇

Work completed on this branch:

  • Isolated concurrency issue with ES Client < 4.1.1
  • Ported, and determined there was no need for the Reconnecting wrapper
  • Determined need for and developed SetQueueTimeoutTo

@bartelink bartelink force-pushed the feature/ges-connector branch from a4f27aa to 60dc6be Compare March 23, 2018 10:24
@bartelink bartelink changed the title Add GES Connector and Load test Add GesConnector and Load test Mar 23, 2018
@bartelink bartelink force-pushed the feature/ges-connector branch from 60dc6be to b4714ed Compare March 27, 2018 07:06
@bartelink bartelink changed the title Add GesConnector and Load test Add Load test; tweak connection settings Mar 27, 2018
@bartelink bartelink force-pushed the feature/ges-connector branch from b4714ed to 778faf4 Compare March 28, 2018 16:02
@bartelink bartelink force-pushed the feature/ges-connector branch from d3bfc2b to 607dc51 Compare April 24, 2018 17:56
@bartelink bartelink force-pushed the feature/ges-connector branch from 2d22801 to e3f701c Compare April 24, 2018 23:19
@bartelink bartelink changed the title Add Load test; tweak connection settings Add Load test; upgrade timeouts and connection settings to 4.1.1-rc1 client Apr 24, 2018
@bartelink bartelink force-pushed the feature/ges-connector branch from e3f701c to 9a8688b Compare April 24, 2018 23:35
@bartelink bartelink force-pushed the feature/ges-connector branch from 9a8688b to 0eada66 Compare April 24, 2018 23:38
@bartelink bartelink changed the title Add Load test; upgrade timeouts and connection settings to 4.1.1-rc1 client Add Load test; Optimize timeouts and connection settings using 4.1.1-rc1 client Apr 24, 2018
@bartelink bartelink merged commit a92ae6c into master Apr 24, 2018
@bartelink bartelink deleted the feature/ges-connector branch April 24, 2018 23:40
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