[IGNITE-15193] Some notifications are lost in configuration listeners.

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.0.0-alpha2
• Fix Version/s: 3.0.0-alpha3
• Labels: iep-55 ignite-3
• Epic Link: Configuration
• Ignite Flags: Docs Required
• Assignee: Ivan Bessonov
• Reporter: Ivan Bessonov
• Reviewer: Semyon Danilov
• Created: 26/Jul/21 10:50
• Updated: 11/Aug/21 08:38
• Resolved: 11/Aug/21 08:38

Description

Consider following test: ConfigurationListenerTest#dataRace()

/** */
@Test
public void dataRace() throws Exception {
    configuration.change(parent -> parent.changeElements(elements ->
        elements.create("name", e -> {}))
    ).get(1, SECONDS);

    List<String> log = new CopyOnWriteArrayList<>();

    configuration.elements().get("name").listen(ctx -> {
        assertNull(ctx.newValue());

        log.add("deleted");

        return completedFuture(null);
    });

    configuration.change(parent -> parent.changeElements(elements ->
        elements.delete("name"))
    ).get(1, SECONDS);

    assertEquals(List.of("deleted"), log);
}

It fails due to wrong configuration access in ConfigurationNotificationsUtil.

Issue Links

• links to: GitHub Pull Request #259