This is an automated email from the ASF dual-hosted git repository.
ni3galave pushed a commit to branch ranger-2.2
in repository https://gitbox.apache.org/repos/asf/ranger.git
The following commit(s) were added to refs/heads/ranger-2.2 by this push:
new 6406df1 RANGER-3342 : Addendum fix Need to make the Ranger embedded server work directory configurable
6406df1 is described below
commit 6406df1c6151aaedeba870e31f50c7100a24d5e0
Author: Vishal Suvagia <vi...@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.");
}