Skip to content

IGNITE-28819 Thin client implementation extraction#13306

Merged
nizhikov merged 237 commits into
apache:masterfrom
nizhikov:IGNITE-28819
Jul 9, 2026
Merged

IGNITE-28819 Thin client implementation extraction#13306
nizhikov merged 237 commits into
apache:masterfrom
nizhikov:IGNITE-28819

Conversation

@nizhikov

@nizhikov nizhikov commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Thank you for submitting the pull request to the Apache Ignite.

In order to streamline the review of the contribution
we ask you to ensure the following steps have been taken:

The Contribution Checklist

  • There is a single JIRA ticket related to the pull request.
  • The web-link to the pull request is attached to the JIRA ticket.
  • The JIRA ticket has the Patch Available state.
  • The pull request body describes changes that have been made.
    The description explains WHAT and WHY was made instead of HOW.
  • The pull request title is treated as the final commit message.
    The following pattern must be used: IGNITE-XXXX Change summary where XXXX - number of JIRA issue.
  • A reviewer has been mentioned through the JIRA comments
    (see the Maintainers list)
  • The pull request has been checked by the Teamcity Bot and
    the green visa attached to the JIRA ticket (see tab PR Check at TC.Bot - Instance 1 or TC.Bot - Instance 2)

Notes

If you need any help, please email dev@ignite.apache.org or ask anу advice on http://asf.slack.com #ignite channel.

nizhikov and others added 30 commits June 25, 2026 16:48
@nizhikov @claude
IGNITE-28819 Scaffold ignite-thin-client-impl module
aaa3483 
Create empty modules/thin-client/impl (ignite-thin-client-impl) for the
thin client implementation extraction. It depends only on ignite-commons,
ignite-binary-api and ignite-thin-client-api (no ignite-core dependency).

Wire it into the reactor (root pom.xml), the BOM dependency management,
and ignite-core (compile dependency + maven-shade include) so the module's
classes are bundled back into ignite-core.jar for binary compatibility,
mirroring ignite-thin-client-api.

Exclude ignite-thin-client-impl from the assembly dependency sets
(dependencies-apache-ignite{,-lgpl,-slim}.xml) as done for
ignite-thin-client-api, since the classes ship inside ignite-core.jar.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ProtocolVersion to ignite-thin-client-impl
8a3c9d2 
Pure relocation (no code change) of the first leaf class of the thin
client implementation. ProtocolVersion has no dependencies on ignite-core
internals and is referenced only from within the thin client package, so
it moves cleanly. Its class still ships in ignite-core.jar via the shade
of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move BinaryNameMapperMode to ignite-thin-client-impl
77e510b 
Pure relocation (no code change) of a leaf class of the thin client
implementation. BinaryNameMapperMode has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ClientCacheEntry to ignite-thin-client-impl
bae4a37 
Pure relocation (no code change) of a leaf class of the thin client
implementation. ClientCacheEntry has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ClientConnection to ignite-thin-client-impl
4302ef4 
Pure relocation (no code change) of a leaf class of the thin client
implementation. ClientConnection has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ClientConnectionStateHandler to ignite-thin-client-…
dd8877d 
…impl

Pure relocation (no code change) of a leaf class of the thin client
implementation. ClientConnectionStateHandler has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ClientJCacheAdapter to ignite-thin-client-impl
7822aab 
Pure relocation (no code change) of a leaf class of the thin client
implementation. ClientJCacheAdapter has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ClientJCacheEntryListenerAdapter to ignite-thin-cli…
5baf3d4 
…ent-impl

Pure relocation (no code change) of a leaf class of the thin client
implementation. ClientJCacheEntryListenerAdapter has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ClientMessageDecoder to ignite-thin-client-impl
09fc180 
Pure relocation (no code change) of a leaf class of the thin client
implementation. ClientMessageDecoder has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ClientMessageHandler to ignite-thin-client-impl
a5ad757 
Pure relocation (no code change) of a leaf class of the thin client
implementation. ClientMessageHandler has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ClientNotificationType to ignite-thin-client-impl
528282d 
Pure relocation (no code change) of a leaf class of the thin client
implementation. ClientNotificationType has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ClientProtocolError to ignite-thin-client-impl
58a2476 
Pure relocation (no code change) of a leaf class of the thin client
implementation. ClientProtocolError has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ClientRetryPolicyContextImpl to ignite-thin-client-…
9c1e1d9 
…impl

Pure relocation (no code change) of a leaf class of the thin client
implementation. ClientRetryPolicyContextImpl has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move IgniteClientFutureImpl to ignite-thin-client-impl
55a5be7 
Pure relocation (no code change) of a leaf class of the thin client
implementation. IgniteClientFutureImpl has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move NotificationListener to ignite-thin-client-impl
a1a6890 
Pure relocation (no code change) of a leaf class of the thin client
implementation. NotificationListener has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ProtocolVersionFeature to ignite-thin-client-impl
e757926 
Pure relocation (no code change) of a leaf class of the thin client
implementation. ProtocolVersionFeature has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move QueryPager to ignite-thin-client-impl
988771f 
Pure relocation (no code change) of a leaf class of the thin client
implementation. QueryPager has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ClientConnectionMultiplexer to ignite-thin-client-impl
5addab3 
Pure relocation (no code change) of a leaf class of the thin client
implementation. ClientConnectionMultiplexer has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ClientInternalBinaryConfiguration to ignite-thin-cl…
7ca591e 
…ient-impl

Pure relocation (no code change) of a leaf class of the thin client
implementation. ClientInternalBinaryConfiguration has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ClientOperation to ignite-thin-client-impl
c42651e 
Pure relocation (no code change) of a leaf class of the thin client
implementation. ClientOperation has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ClientQueryCursor to ignite-thin-client-impl
42004c8 
Pure relocation (no code change) of a leaf class of the thin client
implementation. ClientQueryCursor has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move FieldsQueryPager to ignite-thin-client-impl
83ce436 
Pure relocation (no code change) of a leaf class of the thin client
implementation. FieldsQueryPager has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ProtocolContext to ignite-thin-client-impl
ad76265 
Pure relocation (no code change) of a leaf class of the thin client
implementation. ProtocolContext has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ClientFieldsQueryCursor to ignite-thin-client-impl
ca73b2e 
Pure relocation (no code change) of a leaf class of the thin client
implementation. ClientFieldsQueryCursor has no ignite-core internal dependencies and is
referenced only from within the thin client package. Its class still
ships in ignite-core.jar via the shade of ignite-thin-client-impl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ClientServiceDescriptorImpl to ignite-thin-client-impl
0333a7a 
ClientServiceDescriptorImpl's only ignite-core dependency was the import of
org.apache.ignite.services.Service, used solely in a javadoc {@link}. Per the
move convention, the javadoc-only import is removed (the {@link Service} text
remains; doclint is disabled so it does not fail the build), leaving the class
free of ignite-core internals. Relocated to ignite-thin-client-impl; its class
still ships in ignite-core.jar via the shade.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ClusterNodeFunc to ignite-commons
218ffc5 
Relocate the ClusterNodeFunc utility class from ignite-core to ignite-commons
so it is available to modules that must not depend on ignite-core (the thin
client implementation being extracted uses ClusterNodeFunc.eqNodes). The class
keeps its package org.apache.ignite.internal.util.lang, so all existing
ignite-core call sites are unaffected.

ClusterNodeFunc references four node-id predicates that lived in ignite-core
(ContainsNodeIdsPredicate, EqualsClusterNodeIdPredicate, HasEqualIdPredicate,
HasNotEqualIdPredicate); they depend only on ignite-commons classes
(ClusterNode, S, IgnitePredicate) and are moved alongside it (keeping their
gridfunc package). EqualsUuidPredicate already lives in ignite-commons.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ClientClusterNodeImpl to ignite-thin-client-impl
2853b05 
ClientClusterNodeImpl's only ignite-core dependency was the static import of
ClusterNodeFunc.eqNodes, now resident in ignite-commons. All its other
dependencies (ClusterNode, ClusterMetrics, S, IgniteProductVersion) already
live in ignite-commons, so this is a pure relocation with no code change. The
class still ships in ignite-core.jar via the shade.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move org.apache.ignite.ssl package to ignite-commons
473befd 
Relocate the SSL context factory classes (SslContextFactory,
AbstractSslContextFactory, DelegatingSSLContextSpi, SSLContextWrapper,
SSLServerSocketFactoryWrapper, SSLSocketFactoryWrapper and package-info) from
ignite-core to ignite-commons so they are available to modules that must not
depend on ignite-core. The thin client implementation being extracted uses
SslContextFactory.DFLT_STORE_TYPE / DFLT_KEY_ALGORITHM.

The package org.apache.ignite.ssl is preserved, so all existing call sites
(configuration classes, discovery/communication SPIs, etc.) are unaffected.
The package's only external dependencies are IgniteException and the A
argument-check helper, both already in ignite-commons.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move ClientSslUtils to ignite-thin-client-impl
a7b57cc 
ClientSslUtils's only ignite-core dependencies were the static imports of
SslContextFactory.DFLT_STORE_TYPE / DFLT_KEY_ALGORITHM, now resident in
ignite-commons. Its other dependencies (SslMode, SslProtocol,
ClientConfiguration) already live in ignite-thin-client-api, so this is a pure
relocation with no code change. The class still ships in ignite-core.jar via
the shade.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move AffinityTopologyVersion to ignite-commons
e5ba9db 
Relocate AffinityTopologyVersion from ignite-core to ignite-commons so it is
available to modules that must not depend on ignite-core. The thin client
channel abstraction (ClientChannel#serverTopologyVersion) returns it. The class
is a self-contained value type whose only external dependency (the S
to-string helper) already lives in ignite-commons; its package
org.apache.ignite.internal.processors.affinity is preserved so all existing
call sites are unaffected.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
nizhikov and others added 26 commits July 1, 2026 13:28
@nizhikov @claude
IGNITE-28819 NIO classes use CommonUtils instead of U; move IGNITE_HE…
898a92e 
…ADER/writeFully/bytesEqual to CommonUtils

Relocate IGNITE_HEADER, writeFully(SocketChannel,ByteBuffer) and bytesEqual(...) from
IgniteUtils to CommonUtils (still inherited by IgniteUtils) and repoint the nio package
classes from U to CommonUtils, removing their dependency on core's typedef U.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move IgniteInClosure2X to ignite-commons
595e030 
Its only core tie was an @see CIX2 javadoc reference; drop it (F.wrap is inherited from
commons GridFunc) and move the class to ignite-commons so the NIO communication clients
no longer depend on a core type for it.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move GridWorkerPool and ComputeExecutionRejectedExceptio…
229164d 
…n to ignite-commons

GridWorkerPool uses CommonUtils.cancel/join (already in commons) instead of U, and the
ComputeExecutionRejectedException it throws moves to ignite-commons as well (FQN preserved).
Removes GridNioAsyncNotifyFilter's dependency on a core worker-pool type.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move SpanTransport to ignite-commons; GridNioTracerFilte…
5614fd1 
…r uses SpanManager

Move the self-contained SpanTransport marker interface to ignite-commons and retype
GridNioTracerFilter's tracer to SpanManager (serialize/create are SpanManager methods),
defaulting to NoopSpanManager. Removes the filter's Tracing/NoopTracing/SpanTransport
core dependencies.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 GridSelectorNioSessionImpl resolves trace names via Span…
89c75ea 
…Manager

Pass the SpanManager from GridNioServer into the session and resolve trace names through
SpanManager.traceName instead of the static TraceableMessagesTable.traceName, removing the
session's dependency on the core TraceableMessagesTable.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 GridNioSslFilter takes functional metrics instead of Met…
2cc6db4 
…ricRegistryImpl

Retype the filter's handshake-duration metric to LongConsumer and the rejected-sessions
counter to Runnable; metric creation moves to a new IgniteUtils.sslFilter(...) factory that
the four construction sites now call. Removes GridNioSslFilter's dependency on
MetricRegistryImpl/HistogramMetricImpl/IntMetricImpl.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Decouple GridDirectParser from DirectMessageReader and T…
31ca5c5 
…cpCommunicationSpi

Type the reader as MessageReader (setBuffer/reset suffice, the DirectMessageReader cast was
redundant) and inline the 2-byte direct-type decoding instead of using
TcpCommunicationSpi.makeMessageType.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move the internal.util.nio package to the ignite-nio module
c3c85f2 
Relocate GridNioServer and the entire org.apache.ignite.internal.util.nio package (39
classes, including the ssl subpackage) from ignite-core to the ignite-nio module, now that
the package depends only on ignite-commons, ignite-binary-api and ignite-grid-unsafe. Add
the ignite-grid-unsafe dependency to ignite-nio. ignite-core continues to depend on
ignite-nio, so all existing references resolve.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@nizhikov @claude
IGNITE-28819 Move remaining thin-client implementation classes to ign…
55c0cf8 
…ite-thin-client-impl

With the NIO package now in ignite-nio, add an ignite-nio dependency to ignite-thin-client-impl
and move the last 11 thin-client implementation classes (TcpIgniteClient, TcpClientCache,
ReliableChannelImpl, ClientServicesImpl, the continuous-query/listener classes and the
client-side gridnioserver multiplexer/connection/parser/listener) out of ignite-core.
GridNioClientConnectionMultiplexer builds its SSL filter directly (no metrics) instead of via
the core IgniteUtils.sslFilter factory. ignite-core keeps a compile dependency on
ignite-thin-client-impl, so Ignition and platform-client references still resolve.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@nizhikov
IGNITE-28819 Fix github checks after modules edits
8a4e7d1
@nizhikov
Merge branch 'master' into IGNITE-28819
f836fed
@nizhikov
IGNITE-28819 Merge fixes
4259d77
@nizhikov
Merge branch 'master' into IGNITE-28819
5c3651f
@nizhikov
Merge branch 'master' into IGNITE-28819
6a2beec
@nizhikov
IGNITE-28819 merge
7de32af
@nizhikov
Merge branch 'master' into IGNITE-28819
e704eb9
@nizhikov
IGNITE-28819 Merge fix
c1237b5
@nizhikov
Merge remote-tracking branch 'my/IGNITE-28819' into IGNITE-28819
ff31b5b
@nizhikov
IGNITE-28819 Merge fix
477345f
@nizhikov
IGNITE-28819 Merge fix
cfce923
@nizhikov
Merge branch 'master' into IGNITE-28819
ddcb104
@nizhikov
IGNITE-28819 [4/N] Thin client implementation extraction
11ecf69
@nizhikov
IGNITE-28819 [4/N] Thin client implementation extraction
b514905
@nizhikov
IGNITE-28819 [4/N] Thin client implementation extraction
c3bc1be
@nizhikov
IGNITE-28819 [4/N] Thin client implementation extraction
43d41aa
@nizhikov
IGNITE-28819 [4/N] Thin client implementation extraction
597ccc0
@nizhikov
nizhikov merged commit 32026f4 into apache:master Jul 9, 2026
6 of 7 checks passed
@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown

Possible compatibility issues. Please, check rolling upgrade cases

This PR modifies protected classes (with Order annotation).
Changes to these classes can break rolling upgrade compatibility.

Affected files:

  • modules/core/src/main/java/org/apache/ignite/internal/managers/communication/GridIoMessage.java
  • modules/core/src/test/java/org/apache/ignite/internal/codegen/MessageProcessorTest.java

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@anton-vinogradov anton-vinogradov anton-vinogradov approved these changes

Assignees

No one assigned

Labels

compatibility

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@nizhikov @anton-vinogradov