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

Josh Elser commented on ACCUMULO-3946:
--------------------------------------

Do it at trace, and I think I'm OK with it. I withhold the right to change my 
mind when I see a complete patch :). Our default configs ship with debug on for 
logging so everyone else would be seeing new messages. Make sure to wrap it in 
a conditional too so we don't waste a bunch of cycles creating a String to just 
throw it away if we don't log it:

{code}
if (log.isTraceEnabled()) {
  log.trace(String.format(...));
}
{code}

> Not all accumulo events are audited for Audit logging
> -----------------------------------------------------
>
>                 Key: ACCUMULO-3946
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-3946
>             Project: Accumulo
>          Issue Type: Bug
>    Affects Versions: 1.5.3
>            Reporter: James Mello
>            Assignee: James Mello
>             Fix For: 1.5.4
>
>
> Currently accumulo does not log all the major events such as table creation 
> and permissions changes. Please modify the existing logging to include 
> missing auditing. Note this is related to ticket ACCUMUO-3939.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to