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


Fix it, then Ship it!





agents-audit/src/main/java/org/apache/ranger/audit/provider/BaseAuditHandler.java
Line 338 (original), 338 (patched)
<https://reviews.apache.org/r/73651/#comment312714>

    Consider moving #338 to #308, as these lines are needed only to format 
message used in LOG.debug().


- Madhan Neethiraj


On Oct. 13, 2021, 5:39 a.m., Ramesh Mani wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73651/
> -----------------------------------------------------------
> 
> (Updated Oct. 13, 2021, 5:39 a.m.)
> 
> 
> Review request for ranger, Don Bosco Durai, Abhay Kulkarni, Madhan Neethiraj, 
> Mehul Parikh, Pradeep Agrawal, Selvamohan Neethiraj, Sailaja Polavarapu, and 
> Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-3478
>     https://issues.apache.org/jira/browse/RANGER-3478
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> RANGER-3478:Remove INFO level auditing logs in the Ranger audit frame to 
> reduce verbosity
> 
> 
> Diffs
> -----
> 
>   
> agents-audit/src/main/java/org/apache/ranger/audit/destination/HDFSAuditDestination.java
>  ec9dc4bf6 
>   
> agents-audit/src/main/java/org/apache/ranger/audit/provider/BaseAuditHandler.java
>  19878565c 
>   
> agents-audit/src/main/java/org/apache/ranger/audit/utils/AbstractRangerAuditWriter.java
>  b5967e4f7 
>   
> agents-audit/src/main/java/org/apache/ranger/audit/utils/RangerJSONAuditWriter.java
>  f46e0a578 
> 
> 
> Diff: https://reviews.apache.org/r/73651/diff/1/
> 
> 
> Testing
> -------
> 
> - Verified in local vm with HDFS and HIVE service logs
> 
> 
> Thanks,
> 
> Ramesh Mani
> 
>

Reply via email to