cloud-fan commented on pull request #28629: URL: https://github.com/apache/spark/pull/28629#issuecomment-654887962
Good question. Since we expect end-users to set MDC properties directly, I think MDC properties should take priority over local properties. The local properties is just an internal detail about how we propagate driver side MDC properties to executor side. I think this should be rare as we have the `mdc.` prefix. For example, driver side has a MDC property called `abc`, then we put `mdc.abc` in the local properties. In the executor side, we strip the `mdc.` and put `abc` in the executor MDC. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
