----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62539/ -----------------------------------------------------------
Review request for ranger, Alok Lal, Ankita Sinha, Don Bosco Durai, Colm O hEigeartaigh, Gautam Borad, Madhan Neethiraj, Ramesh Mani, Selvamohan Neethiraj, Velmurugan Periasamy, and Qiang Zhang. Bugs: RANGER-1807 https://issues.apache.org/jira/browse/RANGER-1807 Repository: ranger Description ------- At start() method for RangerHdfsAuthorizer class. isOptimizeSubAccessAuthEnabled() is a static method,it can used directly . {code:title=RangerHdfsAuthorizer.java.java} public void start() { if(LOG.isDebugEnabled()) { LOG.debug("==> RangerHdfsAuthorizer.start()"); } RangerHdfsPlugin plugin = new RangerHdfsPlugin(); plugin.init(); if (plugin.isOptimizeSubAccessAuthEnabled()) { // if (RangerHdfsPlugin.isOptimizeSubAccessAuthEnabled()) { ... ... } public static boolean isOptimizeSubAccessAuthEnabled() { return RangerHdfsPlugin.optimizeSubAccessAuthEnabled; } {code} Diffs ----- hdfs-agent/src/main/java/org/apache/ranger/authorization/hadoop/RangerHdfsAuthorizer.java 97fd5cd5 Diff: https://reviews.apache.org/r/62539/diff/1/ Testing ------- Thanks, wang yuan