[
https://issues.apache.org/jira/browse/IGNITE-22691?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Mikhail Efremov updated IGNITE-22691:
-------------------------------------
Summary: Make startReplica(ReplicationGroupId,
PendingComparableValuesTracker, CompletableFuture) private (was: Delete
startReplica(ReplicationGroupId, PendingComparableValuesTracker,
CompletableFuture))
> Make startReplica(ReplicationGroupId, PendingComparableValuesTracker,
> CompletableFuture) private
> ------------------------------------------------------------------------------------------------
>
> 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
> Time Spent: 10m
> Remaining Estimate: 0h
>
> *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)
