Skip to content

Commit 47447e0

Browse files
jiangzhodongjoon-hyun
authored andcommitted
[SPARK-52679] Revise appender.console.layout.pattern to include MDC keys and thread name
### What changes were proposed in this pull request? This PR updates default log4j properties in Operator helm chart, to make sure it surfaces MDC keys and thread name as needed ### Why are the changes needed? The layout does not include all MDC keys which would provide critical information regarding the resource being reconciled. ### Does this PR introduce _any_ user-facing change? Logging enhancements expected for operator ### How was this patch tested? CIs for lint, and validated logging in dev environments ### Was this patch authored or co-authored using generative AI tooling? No Closes apache#278 from jiangzho/logging. Authored-by: Zhou JIANG <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
1 parent 51c6083 commit 47447e0

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

build-tools/helm/spark-kubernetes-operator/values.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -176,7 +176,7 @@ operatorConfiguration:
176176
appender.console.name = console
177177
appender.console.target = SYSTEM_ERR
178178
appender.console.layout.type = PatternLayout
179-
appender.console.layout.pattern = %d{yy/MM/dd HH:mm:ss} %p %c{1}: %m%n%ex
179+
appender.console.layout.pattern = %d{yy/MM/dd HH:mm:ss} %p %X %C{1.} [%t] %m%n%ex
180180
spark-operator.properties: |+
181181
# Property Overrides. e.g.
182182
# spark.kubernetes.operator.reconciler.intervalSeconds=60

0 commit comments

Comments
 (0)