Ryan van Huuksloot created FLINK-36997:
------------------------------------------
Summary: Add Log4j JSONTemplateLayout to Flink-Dist
Key: FLINK-36997
URL: https://issues.apache.org/jira/browse/FLINK-36997
Project: Flink
Issue Type: Improvement
Affects Versions: 1.20.0
Reporter: Ryan van Huuksloot
Thoughts on including `log4j-layout-template-json` directly into the
flink-dist?
[Source|https://logging.apache.org/log4j/2.x/manual/json-template-layout.html]
As of now, the JSONLayout is deprecated
<https://logging.apache.org/log4j/2.x/manual/layouts.html#JSONLayout>. And
the pattern layout is not meant for production with structured logging
<https://logging.apache.org/log4j/2.x/manual/layouts.html#pattern-layout>.
We provide the other log4j jars
<https://github.com/apache/flink/blob/master/flink-dist/src/main/assemblies/bin.xml#L40-L45>.
Adding the additional dependency means we don't need to maintain versions
across different distributions. Also, at 230KB, it is a tiny dependency
that makes it very easy for users to start using a production grade logging
system.
Based on this thread:
https://lists.apache.org/thread/2qxgpyv8p2dwck6dt8cj020kkkpw8pvh
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
