[ACCUMULO-4588] Simplify Accumulo logging configuration

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.0.0
• Component/s: logger
• Assignee: Unassigned
• Reporter: Mike Walch
• Created: 17/Feb/17 15:48
• Updated: 11/Oct/18 04:01
• Resolved: 05/Jan/18 18:41

Description

The current implementation of Accumulo logging configuration is confusing. There are several configuration files in the conf/ dir and it is hard to know which ones are used in which situation. With the refactoring of Accumulo's scripts in 2.0, it should be easier now to refactor Accumulo's logging. Below are some suggest improvements brought up during code reviews:

Regarding generic_logger.properties...

• Rather than produce these system properties in Accumulo code, we should use the log4j feature to do ${sys:HOSTNAME} (might be log4j2 only, but that's okay... we should switch to that)
• Maybe we shouldn't enable the rolling file appender in the default configs. Users can add that if they want to. A better option would probably be to use the syslog appender by default.

Issue Links

• breaks: ACCUMULO-4649 Testcases in AuditMessageIT fail consistently
• is related to: ACCUMULO-4776 Monitor advertises wrong address in ZooKeeper when binding to all interfaces (0.0.0.0)
• is related to: ACCUMULO-2372 Use POSIX logger for STDERR and STDOUT
• supercedes: ACCUMULO-2823 MonitorLoggingIT can't run on host that is running Monitor role
• links to: GitHub Pull Request #221

Sub-Tasks

• Create a custom log4j AccumuloMonitorAppender: Resolved