accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Josh Elser (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-2373) Daemons should be able to log if log4j.properties specifies sufficient logging
Date Sun, 16 Feb 2014 05:41:19 GMT

    [ 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 <elserj@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)

Mime
View raw message