You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by up...@apache.org on 2015/09/10 01:59:32 UTC

[49/50] [abbrv] incubator-geode git commit: GEODE-311: make pdx logging info level

GEODE-311: make pdx logging info level


Project: http://git-wip-us.apache.org/repos/asf/incubator-geode/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-geode/commit/16571a6f
Tree: http://git-wip-us.apache.org/repos/asf/incubator-geode/tree/16571a6f
Diff: http://git-wip-us.apache.org/repos/asf/incubator-geode/diff/16571a6f

Branch: refs/heads/feature/GEODE-12
Commit: 16571a6f01ba75fa6b8ebe1d80af5a555161859e
Parents: f770621
Author: Darrel Schneider <ds...@pivotal.io>
Authored: Fri Sep 4 13:47:42 2015 -0700
Committer: Darrel Schneider <ds...@pivotal.io>
Committed: Wed Sep 9 13:52:42 2015 -0700

----------------------------------------------------------------------
 .../gemstone/gemfire/pdx/internal/TypeRegistry.java  | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/16571a6f/gemfire-core/src/main/java/com/gemstone/gemfire/pdx/internal/TypeRegistry.java
----------------------------------------------------------------------
diff --git a/gemfire-core/src/main/java/com/gemstone/gemfire/pdx/internal/TypeRegistry.java b/gemfire-core/src/main/java/com/gemstone/gemfire/pdx/internal/TypeRegistry.java
index 4ca1a90..c0bb19c 100644
--- a/gemfire-core/src/main/java/com/gemstone/gemfire/pdx/internal/TypeRegistry.java
+++ b/gemfire-core/src/main/java/com/gemstone/gemfire/pdx/internal/TypeRegistry.java
@@ -122,6 +122,9 @@ public class TypeRegistry {
       if(pdxType != null) {
         this.idToType.put(typeId, pdxType);
         this.typeToId.put(pdxType, typeId);
+        if (logger.isInfoEnabled()) {
+          logger.info("Adding: {}", pdxType.toFormattedString());
+        }
         if (logger.isDebugEnabled()) {
           logger.debug("Adding entry into pdx type registry, typeId: {}  {}", typeId, pdxType);
         }
@@ -186,8 +189,8 @@ public class TypeRegistry {
     if(oldType == null) {
       this.idToType.put(id, newType);
       this.typeToId.put(newType, id);
-      if (logger.isDebugEnabled()) {
-        logger.debug("Defining: {}", newType.toFormattedString());
+      if (logger.isInfoEnabled()) {
+        logger.info("Defining: {}", newType.toFormattedString());
       }
     } else {
       //TODO - this might be overkill, but type definition should be rare enough.
@@ -205,8 +208,8 @@ public class TypeRegistry {
       this.distributedTypeRegistry.addRemoteType(typeId, newType);
       this.idToType.put(typeId, newType);
       this.typeToId.put(newType, typeId);
-      if (logger.isDebugEnabled()) {
-        logger.debug("Adding, from remote WAN: {}", newType.toFormattedString());
+      if (logger.isInfoEnabled()) {
+        logger.info("Adding, from remote WAN: {}", newType.toFormattedString());
       }
     } else {
     //TODO - this might be overkill, but type definition should be rare enough.
@@ -483,8 +486,8 @@ public class TypeRegistry {
     this.distributedTypeRegistry.addImportedType(typeId, importedType);
     this.idToType.put(typeId, importedType);
     this.typeToId.put(importedType, typeId);
-    if (logger.isDebugEnabled()) {
-      logger.debug("Importing type: {}", importedType.toFormattedString());
+    if (logger.isInfoEnabled()) {
+      logger.info("Importing type: {}", importedType.toFormattedString());
     }
   }