Implement node based routing table#6159
Merged
nadav-govari merged 9 commits intonadav/feature-node-based-routingfrom Feb 19, 2026
Merged
Implement node based routing table#6159nadav-govari merged 9 commits intonadav/feature-node-based-routingfrom
nadav-govari merged 9 commits intonadav/feature-node-based-routingfrom
Conversation
nadav-govari
force-pushed
the
nadav/pr2
branch
2 times, most recently
from
6441132 to
745e73e
Compare
Base automatically changed from
nadav/node-affinity-broadcast
to
nadav/feature-node-based-routing
February 17, 2026 16:45
nadav-govari
force-pushed
the
nadav/pr2
branch
from
461f14f to
35a8892
Compare
guilload
reviewed
Feb 18, 2026
Member
guilload
left a comment
guilload
left a comment
There was a problem hiding this comment.
Please address my comments, but good to merge in your other branch.
nadav-govari
merged commit 6b22b1d
into
nadav/feature-node-based-routing
4 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This table will not be in production use as of this PR. It's added alongside the existing one and will be developed incrementally. Once everything is ready, routing will be switched over to this one.
Description
This PR creates a node based routing table alongside the existing one. The new table isn't used yet. This creates the APIs for loading it with data, and for picking nodes, including the power of two choices implementation.
It's added to the router's state, which subscribes to the new IngesterCapacityUpdate broadcast and uses it to update the table entries.
There are additional changes required before this can be landed in production.
How was this PR tested?
New unit tests were added.
These changes are not yet ready for live testing.