tkalkirill commented on code in PR #2871:
URL: https://github.com/apache/ignite-3/pull/2871#discussion_r1405980722
##########
modules/catalog/src/main/java/org/apache/ignite/internal/catalog/commands/CatalogUtils.java:
##########
@@ -399,4 +405,49 @@ static CatalogIndexDescriptor indexOrThrow(Catalog
catalog, int indexId) throws
return index;
}
+
+ /**
+ * Collects all table indexes (including dropped) that the table had in
the requested catalog version range.
+ *
+ * <p>It is expected that at least one version of the catalog contains
table indexes.</p>
+ *
+ * @param catalogService Catalog service.
+ * @param tableId Table ID for which indexes will be collected.
+ * @param catalogVersionFrom Catalog version from which indexes will be
collected (including).
+ * @param catalogVersionTo Catalog version up to which indexes will be
collected (including).
+ * @return Table indexes sorted by {@link CatalogIndexDescriptor#id()}.
+ */
+ public static List<CatalogIndexDescriptor> collectIndexes(
Review Comment:
No, where do you suggest moving it? **TableManager** is already very
overloaded IMHO.
--
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]
