lowka commented on code in PR #4256:
URL: https://github.com/apache/ignite-3/pull/4256#discussion_r1734613284
##########
modules/catalog-compaction/src/main/java/org/apache/ignite/internal/catalog/compaction/CatalogCompactionRunner.java:
##########
@@ -370,12 +394,16 @@ CompletableFuture<Void> propagateTimeToLocalReplicas(long
txBeginTime) {
}, executor);
}
- private long determineLocalMinimumRequiredTime() {
- // TODO https://issues.apache.org/jira/browse/IGNITE-22637 Provide
actual minimum required time.
- return HybridTimestamp.MIN_VALUE.longValue();
- }
private CompletableFuture<Boolean> tryCompactCatalog(Catalog catalog,
LogicalTopologySnapshot topologySnapshot) {
+ for (CatalogIndexDescriptor index : catalog.indexes()) {
+ if (index.status() == CatalogIndexStatus.BUILDING ||
index.status() == CatalogIndexStatus.REGISTERED) {
+ LOG.info("Catalog compaction aborted, index construction is
taking place.");
Review Comment:
Moved code to the `tryCatalogCompaction`.
--
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]
