Hi Community,

Ask: 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.

Thoughts? If there aren't any objections we can open a ticket / PR.

Ryan van Huuksloot
Sr. Production Engineer | Streaming Platform
[image: Shopify]
<https://www.shopify.com/?utm_medium=salessignatures&utm_source=hs_email>

Reply via email to