----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73751/ -----------------------------------------------------------
Review request for ranger. Repository: ranger Description ------- There are some places in the code that check for isInfoEnabled() on loggers. This is superfluous as almost always INFO is enabled and it's also no consistent with the rest of the codebase that does not have these guards. Diffs ----- agents-common/src/main/java/org/apache/ranger/authorization/hadoop/config/RangerConfiguration.java d837a69b6 agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java e998944a5 ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapUserGroupBuilder.java 401b3f01b Diff: https://reviews.apache.org/r/73751/diff/1/ Testing ------- Thanks, David Mollitor
