DiagnosticsAppender for log4j2 master
authorRay Matharu <rmatharu@linkedin.com>
Thu, 17 Jan 2019 22:06:00 +0000 (14:06 -0800)
committerJagadish <jvenkatraman@linkedin.com>
Thu, 17 Jan 2019 22:06:00 +0000 (14:06 -0800)
commitcfedd59715260d8f2d2259ddad2ee1e1f7f28944
treef7c843b5c94de8d8fde586a46b482cf18cd4e436
parent193fd272c931a09305ceced18205e6c4f7c311c7
DiagnosticsAppender for log4j2

This PR adds a DiagnosticsAppender for log4j2.
There exists one already for log4j in samza-log4j.

Two points to note
1. The Appender, LogEvent, and Configuration APIs between log4j2 and log4j are completely different.

2. Log4j requires you to extend the AppenderSkeleton, while log4j2 requires extending the AbstractAppender.

So there was very little overlap (and very little value) in creating a base-class for the two appenders, specially given that java doesn't allow multi-inheritance.

Author: Ray Matharu <rmatharu@linkedin.com>

Reviewers: Pawas C<pchhokra@linkedin.com>

Closes #882 from rmatharu/log4j2appender
samza-log4j2/src/main/java/org/apache/samza/logging/log4j2/SimpleDiagnosticsAppender.java [new file with mode: 0644]