[geode] branch develop updated: log exception first

[bs...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

bschuchardt pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/develop by this push:
     new 671c516  log exception first
     new 43e02ed  Merge pull request #3877 from Bill/feature/GEODE-7012
671c516 is described below

commit 671c516ddb728c02c86c249fb761c57ada486514
Author: Bill Burcham <bb...@pivotal.io>
AuthorDate: Thu Aug 1 13:28:59 2019 -0700

    log exception first
---
 geode-core/src/main/java/org/apache/geode/internal/tcp/Connection.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/geode-core/src/main/java/org/apache/geode/internal/tcp/Connection.java b/geode-core/src/main/java/org/apache/geode/internal/tcp/Connection.java
index 9cde2f5..388874f 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/tcp/Connection.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/tcp/Connection.java
@@ -3152,6 +3152,7 @@ public class Connection implements Runnable {
         // now, so don't let this thread continue.
         throw err;
       } catch (Throwable t) {
+        logger.fatal("Error deserializing message", t);
         // Whenever you catch Error or Throwable, you must also
         // catch VirtualMachineError (see above). However, there is
         // _still_ a possibility that you are dealing with a cascading
@@ -3171,7 +3172,6 @@ public class Connection implements Runnable {
             throw (CancelException) t;
           }
         }
-        logger.fatal("Error deserializing message", t);
       } finally {
         ReplyProcessor21.clearMessageRPId();
       }