sk0x50 commented on code in PR #5297:
URL: https://github.com/apache/ignite-3/pull/5297#discussion_r1971811176
##########
modules/partition-replicator/src/integrationTest/java/org/apache/ignite/internal/partition/replicator/AbstractZoneReplicationTest.java:
##########
@@ -78,11 +81,11 @@
abstract class AbstractZoneReplicationTest extends IgniteAbstractTest {
private static final int BASE_PORT = 20_000;
- protected static final String TEST_ZONE_NAME = "TEST_ZONE";
+ static final String TEST_ZONE_NAME = "TEST_ZONE";
Review Comment:
Let's remove this class and use `ItAbstractColocationTest` instead. It seems
to me, that there is no value in having two base classes that do the same.
--
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
