[
https://issues.apache.org/jira/browse/IGNITE-22691?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Mikhail Efremov updated IGNITE-22691:
-------------------------------------
Description:
*Description*
After IGNITE-21805 there are redundant {{Replica#startReplica}} overload:
{code:java}
public CompletableFuture<Replica> startReplica(
ReplicationGroupId replicaGrpId,
PendingComparableValuesTracker<Long, Void> storageIndexTracker,
CompletableFuture<ReplicaListener> newReplicaListenerFut
) { ... }
{code}
It's marked now as {{@VisibleForTesting}} and {{@Deprecated}} both and its only
purpose is to be used in tests:
*
{{ItRebalanceDistributedTest#verifyThatRaftNodesAndReplicasWereStartedOnlyOnce}}
Checks that the {{startRepica}} and raft-node starts only once. Looks like just
have to add several missing arguments.
* {{ItTxTestCluster#startTable}}
The main goal of this ticket is to delete the method.
*Motivation*
There should the only one public {{Replica#startReplica}} method for
replication group creation.
*Definition of Done*
1. Titled {{Replica#startReplica}} should be deleted.
2. All mentioned tests that called the overloaded method should be fixed in
favor of the single {{Replica#startReplica}} method which is called now in
{{TableManager}}.
was:WIP
> Delete startReplica(ReplicationGroupId, PendingComparableValuesTracker,
> CompletableFuture)
> ------------------------------------------------------------------------------------------
>
> Key: IGNITE-22691
> URL: https://issues.apache.org/jira/browse/IGNITE-22691
> Project: Ignite
> Issue Type: Improvement
> Reporter: Mikhail Efremov
> Assignee: Mikhail Efremov
> Priority: Major
> Labels: ignite-3
>
> *Description*
> After IGNITE-21805 there are redundant {{Replica#startReplica}} overload:
> {code:java}
> public CompletableFuture<Replica> startReplica(
> ReplicationGroupId replicaGrpId,
> PendingComparableValuesTracker<Long, Void> storageIndexTracker,
> CompletableFuture<ReplicaListener> newReplicaListenerFut
> ) { ... }
> {code}
> It's marked now as {{@VisibleForTesting}} and {{@Deprecated}} both and its
> only purpose is to be used in tests:
> *
> {{ItRebalanceDistributedTest#verifyThatRaftNodesAndReplicasWereStartedOnlyOnce}}
> Checks that the {{startRepica}} and raft-node starts only once. Looks like
> just have to add several missing arguments.
> * {{ItTxTestCluster#startTable}}
> The main goal of this ticket is to delete the method.
> *Motivation*
> There should the only one public {{Replica#startReplica}} method for
> replication group creation.
> *Definition of Done*
> 1. Titled {{Replica#startReplica}} should be deleted.
> 2. All mentioned tests that called the overloaded method should be fixed in
> favor of the single {{Replica#startReplica}} method which is called now in
> {{TableManager}}.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
