[GitHub] accumulo issue #221: ACCUMULO-4588 Simplify Accumulo logging configuration

2017-02-24 Thread mikewalch
Github user mikewalch commented on the issue: https://github.com/apache/accumulo/pull/221 > One open-ended question: how can we make sure that log-forwarding works with the removal of MonitorLoggingIT? Do we need to extend MAC to allow us to configure log4j better and test this?

[GitHub] accumulo issue #221: ACCUMULO-4588 Simplify Accumulo logging configuration

2017-02-28 Thread mikewalch
Github user mikewalch commented on the issue: https://github.com/apache/accumulo/pull/221 I would like to merge this today. Let me know if there is anything else that needs to be changed. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] accumulo issue #221: ACCUMULO-4588 Simplify Accumulo logging configuration

2017-02-28 Thread milleruntime
Github user milleruntime commented on the issue: https://github.com/apache/accumulo/pull/221 +1 I'm definitely for logging config improvements. This tends to get over looked on open source projects. --- If your project is set up for it, you can reply to this email and have your repl