tkalkirill commented on code in PR #2448:
URL: https://github.com/apache/ignite-3/pull/2448#discussion_r1299741741
##########
modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/DistributionZoneManager.java:
##########
@@ -202,7 +205,7 @@ public class DistributionZoneManager implements
IgniteComponent {
* Map with states for distribution zones. States are needed to track
nodes that we want to add or remove from the data nodes,
* schedule and stop scale up and scale down processes.
*/
- private final Map<Integer, ZoneState> zonesState;
+ private final Map<Integer, ZoneState> zonesState = new
ConcurrentHashMap<>();
Review Comment:
Perhaps then we need to do this for all fields, such as `busyLock`,
`stopGuard`, `topologyEventListener` and `getMetadataLocallyOnly` inside
`DistributionZoneManager`.
We get a mixed approach, according to my taste, the fewer extra lines of
code the more convenient it is to use, just setting the value of the fields is
such an example.
--
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]
