[
https://issues.apache.org/jira/browse/IGNITE-21202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17807189#comment-17807189
]
Pavel Tupitsyn commented on IGNITE-21202:
-----------------------------------------
As discussed privately with [~v.pyatkov], there is no difference between using
*nodeId* or *consistendId* for partition awareness:
* It is just a way to identify a node among active connections
* If a node restarts, partition assignment change will be detected by
*ClientPrimaryReplicaTracker* and sent to the client
* We use "best effort" mechanism. Client can miss primary replica in many cases
(not all node addresses are known, connection not yet established, assignment
out of date), and the server is still required to handle the request correctly.
* Partition awareness does not apply to explicit tx scenarios (all requests go
to tx coordinator)
Therefore, it does not make sense to rework all 3 clients to use different ids.
I'll remove "deprecated" and close as "won't fix".
> Use node ID instead of node name to identify primary node in client primary
> replica tracker
> -------------------------------------------------------------------------------------------
>
> Key: IGNITE-21202
> URL: https://issues.apache.org/jira/browse/IGNITE-21202
> Project: Ignite
> Issue Type: Improvement
> Components: thin client
> Reporter: Vladislav Pyatkov
> Assignee: Pavel Tupitsyn
> Priority: Major
> Labels: ignite-3
> Fix For: 3.0.0-beta2
>
>
> h3. Motivation
> Recently, we changed the process of granting leases. This process uses the
> node ID as a leaseholder identifier. The other components should also follow
> this consistently.
> h3. Definition of done
> # Here we are using the deprecated property, but we should use the
> leasholder ID.
> {code:java}
> updatePrimaryReplica(tablePartitionId, primaryReplicaEvent.startTime(),
> primaryReplicaEvent.leaseholder()); {code}
> # The lesholder property should be removed from the event parameters.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
