[jira] [Commented] (ACCUMULO-2373) Daemons should be able to log if log4j.properties specifies sufficient logging

["Josh Elser (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ACCUMULO-2373?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13902642#comment-13902642 ] 

Josh Elser commented on ACCUMULO-2373:
--------------------------------------

I believe that this could be fixed by just not starting the daemon when the generic_logger.xml isn't present. When I reset the logging configuration off of ZooKeeper data, I think the DOMConfigurator overrides the log4j.properties that's automatically picked up off of the classpath.

However, I'm not convinced that this is even something that we should bother guarding against. Logging for process in Accumulo is done by the XML log4j configuration files. The example files we provide can be used as-is without any per-system configuration. The monitor is crippled without the log-forwarding enabled. All that considered, I don't see why we should enable users to do this to themselves.

I'll leave this open to see if others have an opinion.

> Daemons should be able to log if log4j.properties specifies sufficient logging
> ------------------------------------------------------------------------------
>
>                 Key: ACCUMULO-2373
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2373
>             Project: Accumulo
>          Issue Type: Bug
>    Affects Versions: 1.5.1
>            Reporter: Vikram Srivastava
>         Attachments: log4j.properties
>
>
> Before this commit:
> commit 0351d0d416df51f0b10d91acdc074dced36e40d4
> Author: Josh Elser <el...@apache.org>
> Date:   Mon Feb 10 23:28:31 2014 -0500
>     ACCUMULO-2334 Remove ACCUMULO_LOG_HOST in favor of pull host and port log-forwarding from zookeeper
> I was able to do logging with only log4j.properties. However, that doesn't work anymore. All my daemons have this as the last line of their logs:
> {noformat}
> 2014-02-14 14:23:38,049 INFO org.apache.accumulo.server.Accumulo: Zookeeper connected and initialized, attemping to talk to HDFS
> 2014-02-14 14:23:38,067 INFO org.apache.accumulo.server.watcher.MonitorLog4jWatcher: Changing monitor log4j address to 10.20.93.170:4560
> 2014-02-14 14:23:38,067 INFO org.apache.accumulo.server.watcher.MonitorLog4jWatcher: Enabled log-forwarding
> {noformat}
> Note that I don't use generic_logger.xml, and this wasn't a requirement for 1.4. But now it seems there is no logging if *_logger.xml files are not present (as ACCUMULO-2349 has also reported).



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)