Skip to content

refactor: centralize console worker connections - #2

Draft
EdsonPetry wants to merge 1 commit into
arrow-58-59-dd-04-08-2026from
agent/console-worker-connector
Draft

refactor: centralize console worker connections#2
EdsonPetry wants to merge 1 commit into
arrow-58-59-dd-04-08-2026from
agent/console-worker-connector

Conversation

@EdsonPetry

Copy link
Copy Markdown
Owner

What changed

  • Introduces a single WorkerConnector abstraction for seed discovery and per-worker polling.
  • Routes reconnects and discovered worker URL parsing through the same connection policy.
  • Preserves the existing plaintext transport and bare-port CLI behavior.

Why

The console previously opened discovery and worker connections through separate code paths. The next PR needs both paths to apply identical logical-origin and physical-dial behavior for Fabric TLS.

Impact

No intended user-visible behavior change. This is the behavior-neutral base of the console stack.

Validation

  • cargo fmt -p datafusion-distributed-console -- --check passed.
  • Full test/clippy build was attempted twice but the local build host exhausted file descriptors while compiling aws-sdk-ec2, including with ulimit -n 8192. CI remains the authoritative check.

Stack

  1. This PR: centralize console connection creation.
  2. Follow-up: support logical TLS worker origins and physical pod dialing.

@EdsonPetry
EdsonPetry changed the base branch from arrow-58-59-dd-04-08-2026 to main August 10, 2026 22:34
@EdsonPetry
EdsonPetry changed the base branch from main to arrow-58-59-dd-04-08-2026 August 10, 2026 22:34
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.

1 participant