Skip to content

QQ: handle tick tweaks #13703

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
Apr 8, 2025
Merged

QQ: handle tick tweaks #13703

merged 1 commit into from
Apr 8, 2025

Conversation

kjnilsson
Copy link
Contributor

@kjnilsson kjnilsson commented Apr 8, 2025

Move leader repair earlier in tick function to ensure more
timely update of meta data store record after leader change.

Also use RPC_TIMEOUT macro for metric/stats multicalls to improve
liveness when a node is connected but partitioned / frozen.

@kjnilsson kjnilsson force-pushed the qq-handle-tick-tweaks branch from 25f8336 to a2dcaaf Compare April 8, 2025 13:39
Move leader repair earlier in tick function to ensure more
timely update of meta data store record after leader change.

Also use RPC_TIMEOUT macro for metric/stats multicalls to improve
liveness when a node is connected but partitioned / frozen.
@kjnilsson kjnilsson force-pushed the qq-handle-tick-tweaks branch from a2dcaaf to 27ef97e Compare April 8, 2025 14:40
@kjnilsson kjnilsson marked this pull request as ready for review April 8, 2025 14:41
@ansd
Copy link
Member

ansd commented Apr 8, 2025

@michaelklishin michaelklishin added this to the 4.1.0 milestone Apr 8, 2025
@michaelklishin michaelklishin merged commit b4fe2cc into main Apr 8, 2025
273 checks passed
@michaelklishin michaelklishin deleted the qq-handle-tick-tweaks branch April 8, 2025 19:58
michaelklishin added a commit that referenced this pull request Apr 8, 2025
acogoluegnes added a commit to rabbitmq/rabbitmq-amqp-java-client that referenced this pull request Apr 9, 2025
github-actions bot pushed a commit to rabbitmq/rabbitmq-amqp-java-client that referenced this pull request Apr 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants