[
https://issues.apache.org/jira/browse/IGNITE-20766?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17789934#comment-17789934
]
Alexander Lapin commented on IGNITE-20766:
------------------------------------------
[~v.pyatkov] LGTM
> Deal with the primary replica provider in integration tests
> -----------------------------------------------------------
>
> Key: IGNITE-20766
> URL: https://issues.apache.org/jira/browse/IGNITE-20766
> Project: Ignite
> Issue Type: Bug
> Reporter: Kirill Tkalenko
> Assignee: Vladislav Pyatkov
> Priority: Major
> Labels: ignite-3
> Time Spent: 40m
> Remaining Estimate: 0h
>
> In the process of implementing IGNITE-20678, it turned out that using
> *TestPlacementDriver* in tests *ItRebalanceDistributedTest* and
> *ItIgniteNodeRestartTest* does not look correct, since for each node the
> primary replica will be defined as the node (primary replica) itself, which
> can lead to unpredictable behavior, we need to figure this out.
> Definition of done
> Both thests have an integration nature; the cluster contains more than one
> node, and the nodes use the network. In these circumstances, the primary
> replica can be changed during an integration test and is not predetermined.
> # I believe the test implementation of the placement drive is not applicable
> here; this is a better way to instance the production implementation. Look at
> the PR attached.
> # Or it would be better to rewrite the test to use the original Ignite node
> instead of the particular instantiated nodes.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
