Skip to content

ra:start_cluster/3: handle timeouts returned by ra_lib:parallel_partition/3 #540

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

Merged
merged 1 commit into from
May 16, 2025

Conversation

michaelklishin
Copy link
Contributor

Closes #539.
References rabbitmq/rabbitmq-server#13828.

@michaelklishin michaelklishin requested a review from kjnilsson May 15, 2025 04:48
@michaelklishin
Copy link
Contributor Author

michaelklishin commented May 15, 2025

@kjnilsson is this what you had in mind?

As for a unit test, it ended up being a change in ra:start_cluster/3 and not ra_lib:partition_parallel/2. It's not obvious to me how to approach testing of the most important part in ra:start_cluster/3.

@michaelklishin michaelklishin added this to the 2.16.9 milestone May 15, 2025
@michaelklishin michaelklishin marked this pull request as ready for review May 16, 2025 05:29
@michaelklishin michaelklishin changed the title ra:start_cluster/3: handle timeouts ra:start_cluster/3: handle timeouts returned by ra_lib:parallel_partition/3 May 16, 2025
@kjnilsson kjnilsson merged commit 0a30d33 into main May 16, 2025
3 checks passed
@michaelklishin michaelklishin deleted the ra-539 branch May 19, 2025 03: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.

Handle parallel computation timeouts when forming a cluster
2 participants