You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by co...@apache.org on 2017/09/25 14:44:19 UTC
ranger git commit: RANGER-1794:The warning information is incorrect
in getRangerServiceByService of ServiceMgr class
Repository: ranger
Updated Branches:
refs/heads/master 99ec61e42 -> 1295e83bd
RANGER-1794:The warning information is incorrect in getRangerServiceByService of ServiceMgr class
Signed-off-by: Colm O hEigeartaigh <co...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/ranger/repo
Commit: http://git-wip-us.apache.org/repos/asf/ranger/commit/1295e83b
Tree: http://git-wip-us.apache.org/repos/asf/ranger/tree/1295e83b
Diff: http://git-wip-us.apache.org/repos/asf/ranger/diff/1295e83b
Branch: refs/heads/master
Commit: 1295e83bd43e86c2ab5868d0fd929cec7b772336
Parents: 99ec61e
Author: peng.jianhua <pe...@zte.com.cn>
Authored: Mon Sep 25 14:58:18 2017 +0800
Committer: Colm O hEigeartaigh <co...@apache.org>
Committed: Mon Sep 25 15:37:26 2017 +0100
----------------------------------------------------------------------
.../src/main/java/org/apache/ranger/biz/ServiceMgr.java | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ranger/blob/1295e83b/security-admin/src/main/java/org/apache/ranger/biz/ServiceMgr.java
----------------------------------------------------------------------
diff --git a/security-admin/src/main/java/org/apache/ranger/biz/ServiceMgr.java b/security-admin/src/main/java/org/apache/ranger/biz/ServiceMgr.java
index 722a566..ce9aa7e 100644
--- a/security-admin/src/main/java/org/apache/ranger/biz/ServiceMgr.java
+++ b/security-admin/src/main/java/org/apache/ranger/biz/ServiceMgr.java
@@ -240,18 +240,19 @@ public class ServiceMgr {
((RangerServiceTag)ret).setTagStore(tagStore);
}
} else {
- LOG.warn("ServiceMgr.getRangerServiceByService(" + service + "): could not find service class '" + serviceDef.getImplClass() + "'");
+ LOG.warn("ServiceMgr.getRangerServiceByService(" + service + "): could not find service class '"
+ + serviceDef.getImplClass() + "' for the service type '" + serviceType + "'");
}
} else {
- LOG.warn("ServiceMgr.getRangerServiceByService(" + service + "): could not find the service-type '" + serviceType + "'");
+ LOG.warn("ServiceMgr.getRangerServiceByService(" + service + "): could not find the service-def for the service type '" + serviceType + "'");
}
} else {
- LOG.warn("ServiceMgr.getRangerServiceByService(" + service + "): could not find the service-type");
+ LOG.warn("ServiceMgr.getRangerServiceByService(" + service + "): could not find the service-type '" + serviceType + "'");
}
if(LOG.isDebugEnabled()) {
LOG.debug("<== ServiceMgr.getRangerServiceByService(" + service + "): " + ret);
- }
+ }
return ret;
}