ibessonov commented on a change in pull request #334:
URL: https://github.com/apache/ignite-3/pull/334#discussion_r708088931
##########
File path:
modules/configuration/src/test/java/org/apache/ignite/internal/configuration/testframework/ConfigurationExtensionTest.java
##########
@@ -50,4 +56,62 @@ public void injectConfiguration(
assertEquals(300, paramCfg.joinTimeout().value());
}
+
+ /** Tests that notifications work on injected configuration instance. */
+ @Test
+ public void notifications() {
+ List<String> log = new ArrayList<>();
+
+ fieldCfg.listen(ctx -> {
+ log.add("update");
Review comment:
Not anymore, but hb rules still apply, collection doesn't have to be
concurrent
--
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]
