Repository: incubator-geode
Updated Branches:
  refs/heads/develop 00de1b0f6 -> 86c89a94d


GEODE-1054: Reduce log level for exeptions thrown from function

When a function exception is thrown when an exception occurs in the function
the results senders are currently logging this at fatal level.  This is not
a "catastrophic" error and should be reduced from fatal to 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/86c89a94
Tree: http://git-wip-us.apache.org/repos/asf/incubator-geode/tree/86c89a94
Diff: http://git-wip-us.apache.org/repos/asf/incubator-geode/diff/86c89a94

Branch: refs/heads/develop
Commit: 86c89a94da97a12deaca6feb0fba4f7b1b60d53b
Parents: 00de1b0
Author: Jason Huynh <[email protected]>
Authored: Wed Mar 9 14:14:35 2016 -0800
Committer: Jason Huynh <[email protected]>
Committed: Wed Mar 23 10:08:33 2016 -0700

----------------------------------------------------------------------
 .../cache/execute/DistributedRegionFunctionResultSender.java       | 2 +-
 .../gemfire/internal/cache/execute/MemberFunctionResultSender.java | 2 +-
 .../cache/execute/PartitionedRegionFunctionResultSender.java       | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/86c89a94/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/execute/DistributedRegionFunctionResultSender.java
----------------------------------------------------------------------
diff --git 
a/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/execute/DistributedRegionFunctionResultSender.java
 
b/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/execute/DistributedRegionFunctionResultSender.java
index 18106c1..79a09de 100644
--- 
a/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/execute/DistributedRegionFunctionResultSender.java
+++ 
b/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/execute/DistributedRegionFunctionResultSender.java
@@ -241,7 +241,7 @@ public final class DistributedRegionFunctionResultSender 
implements
     else {
       ((LocalResultCollector)this.rc).setException(exception);
       //this.lastResult(exception);
-      logger.fatal(LocalizedMessage.create(
+      logger.info(LocalizedMessage.create(
           
LocalizedStrings.DistributedRegionFunctionResultSender_UNEXPECTED_EXCEPTION_DURING_FUNCTION_EXECUTION_ON_LOCAL_NODE),
 exception);
     }
     this.rc.endResults();

http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/86c89a94/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/execute/MemberFunctionResultSender.java
----------------------------------------------------------------------
diff --git 
a/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/execute/MemberFunctionResultSender.java
 
b/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/execute/MemberFunctionResultSender.java
index 30c652c..44126dc 100644
--- 
a/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/execute/MemberFunctionResultSender.java
+++ 
b/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/execute/MemberFunctionResultSender.java
@@ -257,7 +257,7 @@ public final class MemberFunctionResultSender implements 
InternalResultSender {
   public void setException(Throwable exception) {
     ((LocalResultCollector)this.rc).setException(exception);
     //this.lastResult(exception);
-    logger.fatal(LocalizedMessage.create(
+    logger.info(LocalizedMessage.create(
         
LocalizedStrings.MemberResultSender_UNEXPECTED_EXCEPTION_DURING_FUNCTION_EXECUTION_ON_LOCAL_NODE),
 exception);
     this.rc.endResults();
     this.localLastResultRecieved = true;

http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/86c89a94/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/execute/PartitionedRegionFunctionResultSender.java
----------------------------------------------------------------------
diff --git 
a/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/execute/PartitionedRegionFunctionResultSender.java
 
b/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/execute/PartitionedRegionFunctionResultSender.java
index 813d833..12d91f9 100644
--- 
a/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/execute/PartitionedRegionFunctionResultSender.java
+++ 
b/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/execute/PartitionedRegionFunctionResultSender.java
@@ -316,7 +316,7 @@ public final class PartitionedRegionFunctionResultSender 
implements
     }
     else {
       ((LocalResultCollector)this.rc).setException(exception);
-      logger.fatal(LocalizedMessage.create(
+      logger.info(LocalizedMessage.create(
           
LocalizedStrings.PartitionedRegionFunctionResultSender_UNEXPECTED_EXCEPTION_DURING_FUNCTION_EXECUTION_ON_LOCAL_NODE),
           exception);
     }

Reply via email to