Confusing example in Thread Context documentation

[MaxAller]

Description

On this page (in source), this example is given:

LOGGER.debug("Starting background thread for user");

Map<String, String> mdc = ThreadContext.getImmutableContext(); // (1)
List<String> ndc = ThreadContext.getImmutableStack().asList(); // (1)

executor.submit(() -> {
    try (CloseableThreadContext.Instance ignored = CloseableThreadContext
            .putAll(values) // (2)
            .pushAll(messages)) { // (2)

        LOGGER.debug("Processing for user started");
        // ...

    }
});

However...there's no connection between the (1) and (2) items. Is values supposed to be mdc, and messages should be ndc?

[vy]

@MaxAller, you're right, there is room for improvement in that particular example. (Your report indeed qualifies as a documentation bug. I talked with @ppkarwasz, it was a mistake to convert this into a discussion, but... anyway.) Would you mind submitting a PR improving the associated part of the page, please?

[MaxAller]

Can do.

[MaxAller]

Fix merged in #3186