This is an automated email from the ASF dual-hosted git repository. ptupitsyn pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/ignite-3.git
The following commit(s) were added to refs/heads/main by this push: new b768928fed IGNITE-21202 Un-deprecate PrimaryReplicaEventParameters.leaseholder (#3053) b768928fed is described below commit b768928fed89c6529aba6c64391d6c2e35648cdf Author: Pavel Tupitsyn <ptupit...@apache.org> AuthorDate: Tue Jan 16 15:05:22 2024 +0200 IGNITE-21202 Un-deprecate PrimaryReplicaEventParameters.leaseholder (#3053) Using consistentId for partition awareness is ok, we don't need to change it. See detailed explanation in the JIRA comments. --- .../org/apache/ignite/client/handler/ClientPrimaryReplicaTracker.java | 1 - .../internal/placementdriver/event/PrimaryReplicaEventParameters.java | 1 - 2 files changed, 2 deletions(-) diff --git a/modules/client-handler/src/main/java/org/apache/ignite/client/handler/ClientPrimaryReplicaTracker.java b/modules/client-handler/src/main/java/org/apache/ignite/client/handler/ClientPrimaryReplicaTracker.java index 82d4b9955d..9095615e98 100644 --- a/modules/client-handler/src/main/java/org/apache/ignite/client/handler/ClientPrimaryReplicaTracker.java +++ b/modules/client-handler/src/main/java/org/apache/ignite/client/handler/ClientPrimaryReplicaTracker.java @@ -303,7 +303,6 @@ public class ClientPrimaryReplicaTracker implements EventListener<EventParameter TablePartitionId tablePartitionId = (TablePartitionId) primaryReplicaEvent.groupId(); - // TODO: IGNITE-21202 Use the leaseholder ID for thin clients as well. updatePrimaryReplica(tablePartitionId, primaryReplicaEvent.startTime(), primaryReplicaEvent.leaseholder()); return falseCompletedFuture(); // false: don't remove listener. diff --git a/modules/placement-driver-api/src/main/java/org/apache/ignite/internal/placementdriver/event/PrimaryReplicaEventParameters.java b/modules/placement-driver-api/src/main/java/org/apache/ignite/internal/placementdriver/event/PrimaryReplicaEventParameters.java index ba377f043e..3a255edd58 100644 --- a/modules/placement-driver-api/src/main/java/org/apache/ignite/internal/placementdriver/event/PrimaryReplicaEventParameters.java +++ b/modules/placement-driver-api/src/main/java/org/apache/ignite/internal/placementdriver/event/PrimaryReplicaEventParameters.java @@ -66,7 +66,6 @@ public class PrimaryReplicaEventParameters extends CausalEventParameters { } /** Returns leaseholder node consistent ID. */ - @Deprecated public String leaseholder() { return leaseholder; }