tkalkirill commented on code in PR #2741:
URL: https://github.com/apache/ignite-3/pull/2741#discussion_r1374357094
##########
modules/placement-driver-api/src/testFixtures/java/org/apache/ignite/internal/placementdriver/TestPlacementDriver.java:
##########
@@ -37,13 +41,21 @@
@TestOnly
public class TestPlacementDriver extends
AbstractEventProducer<PrimaryReplicaEvent, PrimaryReplicaEventParameters>
implements PlacementDriver {
- private final TestReplicaMetaImpl primaryReplica;
+ private final Supplier<TestReplicaMetaImpl> primaryReplicaSupplier;
Review Comment:
Because, for example, in integration tests, the node ID is assigned only at
the start of the node, and `TestPlacementDriver` is initialized before the
start of the node.
For example see
`org/apache/ignite/internal/replicator/ItPlacementDriverReplicaSideTest.java:161`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@ignite.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
