You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by zh...@apache.org on 2017/04/21 00:33:57 UTC

ranger git commit: RANGER-1526:Error class names in the printed logs

Repository: ranger
Updated Branches:
  refs/heads/master acee1c4f7 -> a3d9b0f91


RANGER-1526:Error class names in the printed logs

Signed-off-by: zhangqiang2 <zh...@zte.com.cn>


Project: http://git-wip-us.apache.org/repos/asf/ranger/repo
Commit: http://git-wip-us.apache.org/repos/asf/ranger/commit/a3d9b0f9
Tree: http://git-wip-us.apache.org/repos/asf/ranger/tree/a3d9b0f9
Diff: http://git-wip-us.apache.org/repos/asf/ranger/diff/a3d9b0f9

Branch: refs/heads/master
Commit: a3d9b0f91d2113a1d1977e6f527d70b76c447f08
Parents: acee1c4
Author: zhangqiang2 <zh...@zte.com.cn>
Authored: Wed Apr 19 17:18:38 2017 +0800
Committer: zhangqiang2 <zh...@zte.com.cn>
Committed: Thu Apr 20 20:32:55 2017 -0400

----------------------------------------------------------------------
 .../org/apache/ranger/biz/ServiceDBStore.java   | 36 ++++++++++----------
 1 file changed, 18 insertions(+), 18 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ranger/blob/a3d9b0f9/security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java
----------------------------------------------------------------------
diff --git a/security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java b/security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java
index 986c37e..33e0e85 100644
--- a/security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java
+++ b/security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java
@@ -283,18 +283,18 @@ public class ServiceDBStore extends AbstractServiceStore {
 	@Override
 	public void init() throws Exception {
 		if (LOG.isDebugEnabled()) {
-			LOG.debug("==> ServiceDefDBStore.init()");
+			LOG.debug("==> ServiceDBStore.init()");
 		}
 
 		if (LOG.isDebugEnabled()) {
-			LOG.debug("<== ServiceDefDBStore.init()");
+			LOG.debug("<== ServiceDBStore.init()");
 		}
 	}
 
 	@PostConstruct
 	public void initStore() {
 		if (LOG.isDebugEnabled()) {
-			LOG.debug("==> ServiceDefDBStore.initStore()");
+			LOG.debug("==> ServiceDBStore.initStore()");
 		}
 
 		if(! legacyServiceDefsInitDone) {
@@ -321,7 +321,7 @@ public class ServiceDBStore extends AbstractServiceStore {
 							}
 						});
 					} catch (Throwable ex) {
-						LOG.fatal("ServiceDefDBStore.initStore(): Failed to update DB: " + ex);
+						LOG.fatal("ServiceDBStore.initStore(): Failed to update DB: " + ex);
 					}
 
 					legacyServiceDefsInitDone = true;
@@ -330,14 +330,14 @@ public class ServiceDBStore extends AbstractServiceStore {
 		}
 
 		if (LOG.isDebugEnabled()) {
-			LOG.debug("<== ServiceDefDBStore.initStore()");
+			LOG.debug("<== ServiceDBStore.initStore()");
 		}
 	}
 
 	@Override
 	public RangerServiceDef createServiceDef(RangerServiceDef serviceDef) throws Exception {
 		if (LOG.isDebugEnabled()) {
-			LOG.debug("==> ServiceDefDBStore.createServiceDef(" + serviceDef + ")");
+			LOG.debug("==> ServiceDBStore.createServiceDef(" + serviceDef + ")");
 		}
 
 		XXServiceDef xServiceDef = daoMgr.getXXServiceDef().findByName(
@@ -575,7 +575,7 @@ public class ServiceDBStore extends AbstractServiceStore {
 		postCreate(createdServiceDef);
 
 		if (LOG.isDebugEnabled()) {
-			LOG.debug("<== ServiceDefDBStore.createServiceDef(" + serviceDef + "): " + createdServiceDef);
+			LOG.debug("<== ServiceDBStore.createServiceDef(" + serviceDef + "): " + createdServiceDef);
 		}
 
 		return createdServiceDef;
@@ -584,7 +584,7 @@ public class ServiceDBStore extends AbstractServiceStore {
 	@Override
 	public RangerServiceDef updateServiceDef(RangerServiceDef serviceDef) throws Exception {
 		if (LOG.isDebugEnabled()) {
-			LOG.debug("==> ServiceDefDBStore.updateServiceDef(" + serviceDef + ")");
+			LOG.debug("==> ServiceDBStore.updateServiceDef(" + serviceDef + ")");
 		}
 
 		Long serviceDefId = serviceDef.getId();
@@ -634,7 +634,7 @@ public class ServiceDBStore extends AbstractServiceStore {
 
 
 		if (LOG.isDebugEnabled()) {
-			LOG.debug("<== ServiceDefDBStore.updateServiceDef(" + serviceDef + "): " + serviceDef);
+			LOG.debug("<== ServiceDBStore.updateServiceDef(" + serviceDef + "): " + serviceDef);
 		}
 
 		return updatedSvcDef;
@@ -1135,7 +1135,7 @@ public class ServiceDBStore extends AbstractServiceStore {
 
 	public void deleteServiceDef(Long serviceDefId, Boolean forceDelete) throws Exception {
 		if (LOG.isDebugEnabled()) {
-			LOG.debug("==> ServiceDefDBStore.deleteServiceDef(" + serviceDefId + ", " + forceDelete + ")");
+			LOG.debug("==> ServiceDBStore.deleteServiceDef(" + serviceDefId + ", " + forceDelete + ")");
 		}
 
 		UserSessionBase session = ContextUtil.getCurrentUserSession();
@@ -1238,7 +1238,7 @@ public class ServiceDBStore extends AbstractServiceStore {
 		postDelete(serviceDef);
 
 		if (LOG.isDebugEnabled()) {
-			LOG.debug("<== ServiceDefDBStore.deleteServiceDef(" + serviceDefId + ", " + forceDelete + ")");
+			LOG.debug("<== ServiceDBStore.deleteServiceDef(" + serviceDefId + ", " + forceDelete + ")");
 		}
 	}
 
@@ -1283,12 +1283,12 @@ public class ServiceDBStore extends AbstractServiceStore {
 	@Override
 	public RangerServiceDef getServiceDef(Long id) throws Exception {
 		if (LOG.isDebugEnabled()) {
-			LOG.debug("==> ServiceDefDBStore.getServiceDef(" + id + ")");
+			LOG.debug("==> ServiceDBStore.getServiceDef(" + id + ")");
 		}
 
 		RangerServiceDef ret = serviceDefService.read(id);
 		if (LOG.isDebugEnabled()) {
-			LOG.debug("<== ServiceDefDBStore.getServiceDef(" + id + "): " + ret);
+			LOG.debug("<== ServiceDBStore.getServiceDef(" + id + "): " + ret);
 		}
 
 		return ret;
@@ -1297,7 +1297,7 @@ public class ServiceDBStore extends AbstractServiceStore {
 	@Override
 	public RangerServiceDef getServiceDefByName(String name) throws Exception {
 		if (LOG.isDebugEnabled()) {
-			LOG.debug("==> ServiceDefDBStore.getServiceDefByName(" + name + ")");
+			LOG.debug("==> ServiceDBStore.getServiceDefByName(" + name + ")");
 		}
 
 		RangerServiceDef ret = null;
@@ -1309,7 +1309,7 @@ public class ServiceDBStore extends AbstractServiceStore {
 		}
 
 		if (LOG.isDebugEnabled()) {
-			LOG.debug("== ServiceDefDBStore.getServiceDefByName(" + name + "): " + ret);
+			LOG.debug("== ServiceDBStore.getServiceDefByName(" + name + "): " + ret);
 		}
 
 		return  ret;
@@ -1357,7 +1357,7 @@ public class ServiceDBStore extends AbstractServiceStore {
 	@Override
 	public RangerService createService(RangerService service) throws Exception {
 		if (LOG.isDebugEnabled()) {
-			LOG.debug("==> ServiceDefDBStore.createService(" + service + ")");
+			LOG.debug("==> ServiceDBStore.createService(" + service + ")");
 		}
 
 		if (service == null) {
@@ -1370,7 +1370,7 @@ public class ServiceDBStore extends AbstractServiceStore {
 		Map<String, String> validConfigs = validateRequiredConfigParams(service, configs);
 		if (validConfigs == null) {
 			if (LOG.isDebugEnabled()) {
-				LOG.debug("==> ConfigParams cannot be null, ServiceDefDBStore.createService(" + service + ")");
+				LOG.debug("==> ConfigParams cannot be null, ServiceDBStore.createService(" + service + ")");
 			}
 			throw restErrorUtil.createRESTException("ConfigParams cannot be null.", MessageEnums.ERROR_CREATING_OBJECT);
 		}
@@ -1485,7 +1485,7 @@ public class ServiceDBStore extends AbstractServiceStore {
 		Map<String, String> validConfigs = validateRequiredConfigParams(service, configs);
 		if (validConfigs == null) {
 			if (LOG.isDebugEnabled()) {
-				LOG.debug("==> ConfigParams cannot be null, ServiceDefDBStore.createService(" + service + ")");
+				LOG.debug("==> ConfigParams cannot be null, ServiceDBStore.createService(" + service + ")");
 			}
 			throw restErrorUtil.createRESTException("ConfigParams cannot be null.", MessageEnums.ERROR_CREATING_OBJECT);
 		}