alievmirza commented on a change in pull request #674:
URL: https://github.com/apache/ignite-3/pull/674#discussion_r822726836
##########
File path:
modules/runner/src/main/java/org/apache/ignite/internal/app/IgniteImpl.java
##########
@@ -332,6 +333,13 @@ public void start(@Nullable String cfg) {
// Deploy all registered watches because all components are ready
and have registered their listeners.
metaStorageMgr.deployWatches();
+ baselineMgr.listenBaselineChange(ctx -> {
Review comment:
I moved it here because this listener must be registered after
`notifyConfigurationListeners();` so we could react to baseline changes only
after proper start. In general, this listener can be registered somewhere in
the init phase, development of which is still in progress.
--
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]
