Review Request 59327: RANGER-1585:Do some code improvement for the error logs in RangerBasePlugin.java

[Qiang Zhang]

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

Review request for ranger, Ankita Sinha, Don Bosco Durai, Colm O hEigeartaigh, 
Gautam Borad, Madhan Neethiraj, Ramesh Mani, Selvamohan Neethiraj, and 
Velmurugan Periasamy.


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


Repository: ranger


Description
-------

Some error logs in RangerBasePlugin.java!
for example:
public void grantAccess(GrantRevokeRequest request, RangerAccessResultProcessor 
resultProcessor) throws Exception {
        if(LOG.isDebugEnabled()) {
                LOG.debug("==> RangerAdminRESTClient.grantAccess(" + request + 
")");
        }
        ...
        if(LOG.isDebugEnabled()) {
                LOG.debug("<== RangerAdminRESTClient.grantAccess(" + request + 
")");
        }
}
we should change 'LOG.debug("==> RangerAdminRESTClient.grantAccess(" + request 
+ ")");' to 'LOG.debug("==> RangerBasePlugin.grantAccess(" + request + ")");'


Diffs
-----

  
agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBasePlugin.java
 13900c6 


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


Testing
-------


Thanks,

Qiang Zhang