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

kfake: add DropControl, SleepOutOfOrder, CoordinatorFor, RehashCoordinators #649

Merged
merged 1 commit into from
Dec 21, 2023

Conversation

twmb
Copy link
Owner

@twmb twmb commented Dec 21, 2023

  • Sleeping was a bit limited because if two requests came in on the same connection, you could not really chain logic. Sleeping out of order allows you to at least run some logic to gate how requests behave with each other. It's not the most obvious, so it is not the default.

  • Adds SleepOutOfOrder

  • Adds CoordinatorFor so you can see which "broker" a coordinator request will go to

  • Adds RehashCoordinators to change where requests are hashed to

The latter two allow you to loop rehashing until a coordinator for your key changes, if you want to force NotCoordinator requests.

…nators

* Sleeping was a bit limited because if two requests came in on the same
  connection, you could not really chain logic. Sleeping out of order
allows you to at least run some logic to gate how requests behave with
each other. It's not the most obvious, so it is not the default.

* Adds SleepOutOfOrder

* Adds CoordinatorFor so you can see which "broker" a coordinator
  request will go to

* Adds RehashCoordinators to change where requests are hashed to

The latter two allow you to loop rehashing until a coordinator for your
key changes, if you want to force NotCoordinator requests.
@twmb twmb merged commit 2907ba9 into master Dec 21, 2023
6 checks passed
@twmb twmb deleted the kfake-control-sleep-more branch December 21, 2023 02:17
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.

1 participant