[ACCUMULO-4409] Create a custom log4j AccumuloMonitorAppender

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.0.0
• Labels: release_notes
• Assignee: Christopher Tubbs
• Reporter: Christopher Tubbs
• Created: 17/Aug/16 02:56
• Updated: 13/Oct/17 03:24
• Resolved: 13/Oct/17 03:24

Description

The current AsyncSocketAppender was an improvement over the older log4j configuration options, but it did not fully satisfy the description in ACCUMULO-2343.

What we really need is an appender which can be managed by configuration alone, without using code to reload it with new system properties when the location of the monitor changes.

So, rather than having a ZK watcher reload the log4j system, the appender itself should internally detect the relocation of the monitor and adjust its behavior accordingly. That way, the configuration files do not need to refer to system properties for the hostname/port of the monitor, and there is no need to reload the log4j logging when things change.

This will allow us to remove more custom log4j code, which gets us closer to users having the ability to use another implementation for the slf4j API (though this feature of logging to the monitor might not be available for other slf4j implementations, unless they support using log4j appenders).

Issue Links

• is related to: ACCUMULO-4599 Simplify how accumulo runs client jars
• relates to: ACCUMULO-2343 Create Appender specifically for log-forwarding
• links to: GitHub Pull Request #223