Skip to content
This repository has been archived by the owner on Nov 28, 2022. It is now read-only.

Enhance integration tests to do multiple account create/lookups to crash the server. #10

Closed
wants to merge 1 commit into from
Closed

Conversation

nurturenature
Copy link

Please see issue #9.

This PR updates the integration tests to loop through account creation/lookup and demonstrate the issue.

After ~10 operations the server will crash and the client will hang.

@batiati
Copy link
Collaborator

batiati commented Sep 26, 2022

Hi @nurturenature thank you for taking tigerbeetle-java for a spin!
I found your jepsen-tigerbeetle quite interesting, awesome work!

Regarding your PR, we are still working at our storage layer, so the tests are artificially limited to the minimum amount of transactions needed just to verify the API's correctness.
But don't worry, we have a large set of stress tests planned that will test much more realistic use cases.

@nurturenature
Copy link
Author

Thanks for the update!

I've updated the jepsen-tigerbeetle notes to indicate that it's an intentional API limitation while the storage layer is being developed vs a bug.

As the behavior is intentional for this stage of development, will close this pr.

Am looking forward to experimenting with Jepsen tests as things progress.

@jorangreef
Copy link
Member

Thanks @nurturenature this was awesome for us to see!

We'll keep you posted and looking forward to your experiments with Jepsen soon.

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.

3 participants