Skip to content

Commit 4324afd

Browse files
committed
IGNITE-26737 Deprecate non-colocation mode in 3.1 (#6797)
1 parent 7a99db4 commit 4324afd

File tree

2 files changed

+13
-2
lines changed

2 files changed

+13
-2
lines changed

modules/core/src/main/java/org/apache/ignite/internal/lang/IgniteSystemProperties.java

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,12 @@ public final class IgniteSystemProperties {
4444
public static final String IGNITE_SKIP_STORAGE_UPDATE_IN_BENCHMARK = "IGNITE_SKIP_STORAGE_UPDATE_IN_BENCHMARK";
4545

4646
// TODO https://issues.apache.org/jira/browse/IGNITE-22522 Remove this feature flag.
47-
/** Enables zone based replication (aka colocation) feature. */
47+
/**
48+
* Enables zone based replication (aka colocation) feature.
49+
*
50+
* @deprecated Non-colocation mode is planned to be removed in version 3.2.
51+
*/
52+
@Deprecated(since = "3.1", forRemoval = true)
4853
public static final String COLOCATION_FEATURE_FLAG = "IGNITE_ZONE_BASED_REPLICATION";
4954

5055
/**
@@ -60,8 +65,10 @@ private IgniteSystemProperties() {
6065
*
6166
* <p>Do not use in production code (apart from {@link NodeProperties} implementations). If a component needs colocation status,
6267
* it should get one from {@link NodeProperties}.
68+
*
69+
* @deprecated Non-colocation mode is planned to be removed in version 3.2.
6370
*/
64-
@Deprecated
71+
@Deprecated(since = "3.1", forRemoval = true)
6572
public static boolean colocationEnabled() {
6673
return getBoolean(COLOCATION_FEATURE_FLAG, true);
6774
}

modules/runner/src/main/java/org/apache/ignite/internal/app/NodePropertiesImpl.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,6 +89,10 @@ private void detectAndSaveColocationStatusIfNeeded() {
8989
IgniteSystemProperties.colocationEnabled()
9090
);
9191
}
92+
if (!colocationEnabled) {
93+
LOG.warn("Zone based replication is disabled, this mode is deprecated and will be removed in version 3.2. "
94+
+ "Consider migrating to zone based replication (which is default now).");
95+
}
9296
}
9397

9498
private void saveToVault(boolean enablementStatus) {

0 commit comments

Comments
 (0)