sergeyuttsel commented on code in PR #1968:
URL: https://github.com/apache/ignite-3/pull/1968#discussion_r1182524793
##########
modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/DistributionZoneManager.java:
##########
@@ -161,6 +171,17 @@ public class DistributionZoneManager implements
IgniteComponent {
/** The logger. */
private static final IgniteLogger LOG =
Loggers.forClass(DistributionZoneManager.class);
+ /**
+ * If this property is set to {@code true} then an attempt to get the
configuration property directly from Meta storage will be skipped,
+ * and the local property will be returned.
+ * TODO: IGNITE-16774 This property and overall approach, access
configuration directly through Meta storage,
+ * TODO: will be removed after fix of the issue.
+ */
+ private final boolean getMetadataLocallyOnly =
IgniteSystemProperties.getBoolean("IGNITE_GET_METADATA_LOCALLY_ONLY");
+
+ /** Versioned store for zones id. */
+ private final IncrementalVersionedValue<Set<Integer>> zonesByIdVv;
Review Comment:
Thanks. I've removed VersiondValue and used ConfigurationNamedListListener
to await the zone is created locally. I discussed this approach with Ivan
Bessonov and he confirmed that it is possible
to invoke NamedConfigurationTree#stopListenElements in the listener in this
case.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
