rpuch commented on code in PR #5230:
URL: https://github.com/apache/ignite-3/pull/5230#discussion_r1959925143
##########
modules/core/src/main/java/org/apache/ignite/internal/lang/IgniteSystemProperties.java:
##########
@@ -79,10 +79,14 @@ public final class IgniteSystemProperties {
/** Skip storage update in a benchmark. */
public static final String IGNITE_SKIP_STORAGE_UPDATE_IN_BENCHMARK =
"IGNITE_SKIP_STORAGE_UPDATE_IN_BENCHMARK";
- // TODO https://issues.apache.org/jira/browse/IGNITE-22115 Remove this
feature flag.
+ // TODO https://issues.apache.org/jira/browse/IGNITE-22522 Remove this
feature flag.
/** Enables zone based replication (aka colocation) feature. */
public static final String COLOCATION_FEATURE_FLAG =
"IGNITE_ZONE_BASED_REPLICATION";
+ // TODO https://issues.apache.org/jira/browse/IGNITE-22522 Remove.
+ /* Feature flag for zone based collocation track */
+ public static volatile boolean ENABLED_COLOCATION_DEFAULT =
getBoolean(COLOCATION_FEATURE_FLAG, false);
Review Comment:
Agh, in this implementation probably `ENABLED_COLOCATION` would be a better
name. I was thinking about something different, but your approach is better,
but the name is a bit off.
--
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
