GitHub user mal opened a pull request:
https://github.com/apache/storm/pull/2781
STORM-3163: Make ShellBolt logger setup calls occur in-thread to support
MDC (BACKPORT)
Backport to 1.x.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mal/storm STORM-3163-1.x-branch
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/2781.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2781
----
commit cb008f89502b3891d2c23f534fd7d6bb81fc0e83
Author: Mal Graty <mal.graty@...>
Date: 2018-07-29T17:22:43Z
Setup ShellBolt logger in the thread using it
Prior to this the logger set up would occur in a separate thread (the
parent), to the one doing the logging. This meant that MDC values, which
are thread local, could not persist from the set up phase. By doing set
up in the same thread, MDC values are now reliably available during log
calls.
----
---
