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.");
                             }

Reply via email to