You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ignite.apache.org by "ygerzhedovich (via GitHub)" <gi...@apache.org> on 2023/09/25 14:35:13 UTC

[GitHub] [ignite-3] ygerzhedovich commented on a diff in pull request #2619: IGNTIE-20339 Get rid of legacy IndexEvent, TableEvent and SchemaEvent

ygerzhedovich commented on code in PR #2619:
URL: https://github.com/apache/ignite-3/pull/2619#discussion_r1335980511


##########
modules/table/src/main/java/org/apache/ignite/internal/table/distributed/TableManager.java:
##########
@@ -1329,10 +1316,11 @@ private CompletableFuture<Void> createTableLocally(
         tablesById(causalityToken).thenRun(() -> inBusyLock(busyLock, () -> completeApiCreateFuture(table)));
 
         // TODO should be reworked in IGNITE-16763
-        // We use the event notification future as the result so that dependent components can complete the schema updates.
+        // We used the event notification future as the result so that dependent components can complete the schema updates.

Review Comment:
   what event do you mena here?



-- 
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-unsubscribe@ignite.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org