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

(Updated Aug. 2, 2016, 2:07 a.m.)


Review request for ranger.


Bugs: RANGER-1131
    https://issues.apache.org/jira/browse/RANGER-1131


Repository: ranger


Description
-------

AuditHandler.log function returns true or false to indicate the audit log 
result, but the caller doesn't handle this return value. So if the audit is 
failed to log, we can't find any clue in the trace.


Diffs (updated)
-----

  
agents-common/src/main/java/org/apache/ranger/plugin/audit/RangerDefaultAuditHandler.java
 60d3b0c 

Diff: https://reviews.apache.org/r/50649/diff/


Testing (updated)
-------

Warn level trace is added to indicate audit logging failure. The trace can be 
found in the component who has ranger plugin enabled.
Built clean and ran all unit tests passed.


Thanks,

Dongying Jiao

Reply via email to