You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ignite.apache.org by pt...@apache.org on 2024/01/16 13:05:28 UTC
(ignite-3) branch main updated: IGNITE-21202 Un-deprecate PrimaryReplicaEventParameters.leaseholder (#3053)
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 <pt...@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;
}