tkalkirill commented on code in PR #5020:
URL: https://github.com/apache/ignite-3/pull/5020#discussion_r1912910453
##########
modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/DistributionZoneManager.java:
##########
@@ -278,6 +278,8 @@ public DistributionZoneManager(
@Override
public CompletableFuture<Void> startAsync(ComponentContext
componentContext) {
return inBusyLockAsync(busyLock, () -> {
+ partitionDistributionResetTimeoutConfiguration.init();
Review Comment:
> This fix is about the wrong assert, but when I fixed it I found out the
issue with
ItHighAvailablePartitionsRecoveryTest#testHaRecoveryOnZoneTimersRestoreAfterNodeRestart,
because assertion starts to work. So, this test fixed here also.
Okay, please indicate this in the ticket description so that it is clearer
which test checks this.
If I understand correctly, this ticket should fix the issue from
[IGNITE-24057](https://issues.apache.org/jira/browse/IGNITE-24057)? If so, then
without a test this assumption may be wrong and it is better to make sure by
writing a test.
--
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]
