Skip to content

IGNITE-27521 Java client: handle multiple endpoints for the same node#7745

Open
ptupitsyn wants to merge 12 commits intoapache:mainfrom
ptupitsyn:ignite-27521
Open

IGNITE-27521 Java client: handle multiple endpoints for the same node#7745
ptupitsyn wants to merge 12 commits intoapache:mainfrom
ptupitsyn:ignite-27521

Conversation

@ptupitsyn
Copy link
Contributor

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR addresses IGNITE-27521 by improving the Java thin client behavior when multiple distinct endpoints resolve to the same server node, adding a warning and extending test coverage for the scenario.

Changes:

  • Add a warning log when different endpoints connect to the same server node.
  • Update connections() reporting to reflect multiple active endpoint connections to the same node (while skipping duplicate-config weights).
  • Add a new DNS discovery test that asserts the warning is logged and that both connections are tracked and closed.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
modules/client/src/test/java/org/apache/ignite/client/ClientDnsDiscoveryTest.java Adds a new test for “multiple endpoints -> same node” and wires a test logger into client configuration.
modules/client/src/main/java/org/apache/ignite/internal/client/ReliableChannel.java Emits a warning on multi-endpoint-to-same-node detection and adjusts connections() to enumerate per-endpoint holders.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ptupitsyn ptupitsyn requested a review from isapego March 11, 2026 06:20
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.

2 participants