sanpwc commented on code in PR #2183: URL: https://github.com/apache/ignite-3/pull/2183#discussion_r1229137383
########## modules/metastorage-api/src/main/java/org/apache/ignite/internal/metastorage/MetaStorageManager.java: ########## @@ -170,8 +170,10 @@ public interface MetaStorageManager extends IgniteComponent { * Starts all registered watches. * * <p>Should be called after all Ignite components have registered required watches and they are ready to process Meta Storage events. + * + * @return Future to complete. Review Comment: You're joking right? The core idea of the current ticket (IGNITE-19606) is to linearise ms.start() and ms.deployWatches(). Do we still have an implementation with unconnected start and deployWatches? -- 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