kgusakov commented on code in PR #5020:
URL: https://github.com/apache/ignite-3/pull/5020#discussion_r1913136054


##########
modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/DistributionZoneManager.java:
##########
@@ -301,8 +303,6 @@ public CompletableFuture<Void> startAsync(ComponentContext 
componentContext) {
             // fires CatalogManager's ZONE_CREATE event, and the state of 
DistributionZoneManager becomes consistent.
             int catalogVersion = catalogManager.latestCatalogVersion();
 
-            partitionDistributionResetTimeoutConfiguration.init();

Review Comment:
   Current holder is a tiny wrapeer for one caller around one configuration 
property. Nothing more. We already have a discussion with Ivan about the 
extension to more generic mechanism with the listeners aggregation and etc. - I 
think we should do it there



-- 
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]

Reply via email to