[jira] [Comment Edited] (SENTRY-346) Create new FileAppender used in log4j to keep all the logs

["Colin Ma (JIRA)" <ji...@apache.org>]

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

Colin Ma edited comment on SENTRY-346 at 7/22/14 1:58 AM:
----------------------------------------------------------

I think audit log is for some DDL command(eg, create role, drop role, grant, revoke), and there won't be many logs generated by sentry,
Currently, RollingFileAppender in log4j can be configured like:
log4j.appender.auditLog.MaxFileSize=10MB
log4j.appender.auditLog.MaxBackupIndex=1000
There will be 10GB audit log. Is the storage enough for the audit log?  
I'm wondering if the new FileAppender is necessary.
Any comments are welcome, thanks.


was (Author: colinma):
In my mide audit log is for some DDL command(eg, create role, drop role, grant, revoke), and there won't be many logs generated by sentry,
Currently, RollingFileAppender in log4j can be configured like:
log4j.appender.auditLog.MaxFileSize=10MB
log4j.appender.auditLog.MaxBackupIndex=1000
There will be 10GB audit log. Is the storage enough for the audit log?  
I'm wondering if the new FileAppender is necessary.
Any comments are welcome, thanks.

> Create new FileAppender used in log4j to keep all the logs
> ----------------------------------------------------------
>
>                 Key: SENTRY-346
>                 URL: https://issues.apache.org/jira/browse/SENTRY-346
>             Project: Sentry
>          Issue Type: Sub-task
>    Affects Versions: 1.4.0
>            Reporter: Colin Ma
>            Assignee: Colin Ma
>         Attachments: sentry-346.v1.patch
>
>
> The audit log will be generated by log4j. Currently, the RollingFileAppender deletes logs after rolling over certain number of time(according to MaxBackupIndex property). In order to keep all audit logs, there need a new FileAppender.



--
This message was sent by Atlassian JIRA
(v6.2#6252)