You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sentry.apache.org by Arjun Mishra <am...@cloudera.com> on 2017/07/26 16:07:24 UTC

Review Request 61141: SENTRY-1867 - DataNucleus.Query INFO level logging spams Sentry log files

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61141/
-----------------------------------------------------------

Review request for sentry, Alexander Kolbasov and Vadim Spector.


Repository: sentry


Description
-------

In the Sentry service, DataNucleus.Query property, when not defined in log4j.properties file, is by default set to INFO. This floods the log files with INFO level logs every few seconds. We should set the default value to WARN to avoid this, unless until specified in the log4j.properties file


Diffs
-----

  sentry-core/sentry-core-common/src/main/java/org/apache/sentry/SentryMain.java 3a981b2a1 


Diff: https://reviews.apache.org/r/61141/diff/1/


Testing
-------


Thanks,

Arjun Mishra


Re: Review Request 61141: SENTRY-1867 - DataNucleus.Query INFO level logging spams Sentry log files

Posted by Vamsee Yarlagadda <va...@cloudera.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61141/#review181471
-----------------------------------------------------------


Ship it!




Ship It!

- Vamsee Yarlagadda


On July 26, 2017, 4:07 p.m., Arjun Mishra wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61141/
> -----------------------------------------------------------
> 
> (Updated July 26, 2017, 4:07 p.m.)
> 
> 
> Review request for sentry, Alexander Kolbasov and Vadim Spector.
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> In the Sentry service, DataNucleus.Query property, when not defined in log4j.properties file, is by default set to INFO. This floods the log files with INFO level logs every few seconds. We should set the default value to WARN to avoid this, unless until specified in the log4j.properties file
> 
> 
> Diffs
> -----
> 
>   sentry-core/sentry-core-common/src/main/java/org/apache/sentry/SentryMain.java 3a981b2a1 
> 
> 
> Diff: https://reviews.apache.org/r/61141/diff/1/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Arjun Mishra
> 
>