denis-chudov commented on code in PR #5092:
URL: https://github.com/apache/ignite-3/pull/5092#discussion_r1960114562
##########
modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/DistributionZoneManager.java:
##########
@@ -1620,44 +710,41 @@ private void
registerCatalogEventListenersOnStartManagerBusy() {
catalogManager.listen(ZONE_ALTER, new
ManagerCatalogAlterZoneEventListener());
}
- private CompletableFuture<Void> createOrRestoreZonesStates(long
recoveryRevision, int catalogVersion) {
- List<CompletableFuture<Void>> futures = new ArrayList<>();
-
- // TODO: IGNITE-20287 Clean up abandoned resources for dropped tables
from vault and metastore
- for (CatalogZoneDescriptor zone :
catalogManager.catalog(catalogVersion).zones()) {
- futures.add(restoreZoneStateBusy(zone, recoveryRevision));
- }
-
- return allOf(futures.toArray(CompletableFuture[]::new));
- }
-
/**
* Restore the event of the updating the logical topology from Meta
Storage, that has not been completed before restart.
* Also start scale up/scale down timers.
*
* @param recoveryRevision Revision of the Meta Storage after its recovery.
* @return Future that represents the pending completion of the operations.
*/
- private CompletableFuture<Void>
restoreLogicalTopologyChangeEventAndStartTimers(long recoveryRevision, int
catalogVersion) {
+ private CompletableFuture<Void>
restoreLogicalTopologyChangeEventAndStartTimers(long recoveryRevision) {
Review Comment:
renamed and fixed the javadoc.
--
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
