I've opened a ticket and a PR. Ticket: https://issues.apache.org/jira/browse/FLINK-36997 PR: https://github.com/apache/flink/pull/25888
It should be easy to include in the Flink 2.0 release. If someone could help with review that would be great. Thanks, Ryan van Huuksloot Sr. Production Engineer | Streaming Platform [image: Shopify] <https://www.shopify.com/?utm_medium=salessignatures&utm_source=hs_email> On Mon, Dec 16, 2024 at 4:26 PM Ryan van Huuksloot < ryan.vanhuuksl...@shopify.com> wrote: > 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> >
