Skip to content

Conversation

@kjnilsson
Copy link
Contributor

@kjnilsson kjnilsson commented Mar 27, 2025

Ra improvements:

  • Don't allow a non-voter to start elections
  • Register with ra directory before initialising ra server.
  • Trigger tick_timeout immediately after entering leader state.
  • Set a configurable segment max size

This commit also includes a change to turn the quorum queue
become leader callback to become a noop and instead rely on
the more promptly tick_handler to handle the meta data store
update after a leader election.

This more prompt tick update means there should be a much shorter
gap between the queue metrics being deleted from the old leader
node to them being available again on the new node resulting
in smoother message count metrics.

Fix test that relied on waiting on too simplistic a property
before asserting.

@mergify mergify bot added the make label Mar 27, 2025
@kjnilsson kjnilsson marked this pull request as ready for review March 27, 2025 16:45
Ra improvements:

* Don't allow a non-voter to start elections
* Register with ra directory before initialising ra server.
* Trigger tick_timeout immediately after entering leader state.
* Set a configurable segment max size

This commit also includes a change to turn the quorum queue
become leader callback to become a noop and instead rely on
the more promptly tick_handler to handle the meta data store
update after a leader election.

This more prompt tick update means there should be a much shorter
gap between the queue metrics being deleted from the old leader
node to them being available again on the new node resulting
in smoother message count metrics.

Fix test that relied on waiting on too simplistic a property
before asserting.
@michaelklishin michaelklishin merged commit 860bb7c into main Mar 27, 2025
273 checks passed
@michaelklishin michaelklishin deleted the ra-2.16.5 branch March 27, 2025 18:33
michaelklishin added a commit that referenced this pull request Mar 27, 2025
Ra 2.16.5 - bug fixes and minor improvements (backport #13638)
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