Skip to content

control-connection: keep zero-token hosts#844

Closed
dkropachev wants to merge 2 commits into
masterfrom
fix/zero-token-host-refresh
Closed

control-connection: keep zero-token hosts#844
dkropachev wants to merge 2 commits into
masterfrom
fix/zero-token-host-refresh

Conversation

@dkropachev

Copy link
Copy Markdown
Collaborator

Summary

  • keep zero-token nodes as topology members during control connection metadata refresh
  • omit zero-token hosts from the token map
  • expand unit coverage for system.peers, system.peers_v2, local row metadata, and empty token list shapes

Fixes #843

Driver surface

Control connection topology refresh and token metadata population.

Protocol or compatibility risks

Low. Token map behavior stays unchanged for hosts without tokens; the topology refresh no longer filters valid zero-token hosts from metadata.

Tests

  • repo-ci fast: 044b625db0332d88a9741c3a707708b73f5d1b53dd140745f211e542e3cb0996

Integration scenarios

Not required; this is unit-covered metadata refresh behavior.

@dkropachev

Copy link
Copy Markdown
Collaborator Author

Withdrawing. This PR assumed zero-token nodes should remain in driver topology metadata while being omitted from the token map. That is likely the wrong model here: driver known hosts are query/pool targets, and zero-token nodes should not be added if the driver cannot execute on them.

@dkropachev dkropachev closed this May 5, 2026
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.

Control connection ignores zero-token nodes during metadata refresh

1 participant