[
https://issues.apache.org/jira/browse/IGNITE-21202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17807193#comment-17807193
]
Vladislav Pyatkov commented on IGNITE-21202:
--------------------------------------------
LGTM
> 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
>
> Time Spent: 10m
> Remaining Estimate: 0h
>
> 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)
