This is an automated email from the ASF dual-hosted git repository. mehul pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/ranger.git
The following commit(s) were added to refs/heads/master by this push: new 83e3b19 RANGER-3342 : Addendum fix Need to make the Ranger embedded server work directory configurable 83e3b19 is described below commit 83e3b19c5c5ec453a875915b73f80a1326f01631 Author: Vishal Suvagia <vishalsuva...@apache.org> AuthorDate: Sat Jul 24 00:43:37 2021 +0530 RANGER-3342 : Addendum fix Need to make the Ranger embedded server work directory configurable Issue: Earlier fix included a Logger check for Log Level Fine, but the LOG get level is found to be null leading to a NullPointerException. Changes: Updated the check for required LOG level to be Fine. Testing: Validated on a cluster. Signed-off-by: Mehul Parikh <me...@apache.org> --- .../src/main/java/org/apache/ranger/server/tomcat/EmbeddedServer.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/embeddedwebserver/src/main/java/org/apache/ranger/server/tomcat/EmbeddedServer.java b/embeddedwebserver/src/main/java/org/apache/ranger/server/tomcat/EmbeddedServer.java index d3b1084..62a188b 100644 --- a/embeddedwebserver/src/main/java/org/apache/ranger/server/tomcat/EmbeddedServer.java +++ b/embeddedwebserver/src/main/java/org/apache/ranger/server/tomcat/EmbeddedServer.java @@ -266,7 +266,7 @@ public class EmbeddedServer { if (!workDirPath.isEmpty() && new File(workDirPath).exists()) { webappCtx.setWorkDir(workDirPath); } else { - if (LOG.getLevel().equals(Level.FINE)) { + if (LOG.isLoggable(Level.FINE)) { LOG.fine("Skipping to set tomcat server work directory, '" + workDirPath + "', as it is blank or directory does not exist."); }