svn commit: r1447601 - /hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/RetryingHMSHandler.java

[namit]

Author: namit
Date: Tue Feb 19 06:48:07 2013
New Revision: 1447601

URL: http://svn.apache.org/r1447601
Log:
HIVE-4023 Improve Error Logging in MetaStore
(Bhushan Mandhani via namit)


Modified:
    
hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/RetryingHMSHandler.java

Modified: 
hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/RetryingHMSHandler.java
URL: 
http://svn.apache.org/viewvc/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/RetryingHMSHandler.java?rev=1447601&r1=1447600&r2=1447601&view=diff
==============================================================================
--- 
hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/RetryingHMSHandler.java
 (original)
+++ 
hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/RetryingHMSHandler.java
 Tue Feb 19 06:48:07 2013
@@ -114,10 +114,12 @@ public class RetryingHMSHandler implemen
             caughtException = e.getCause();
           }
           else {
+            LOG.error(ExceptionUtils.getStackTrace(e.getCause()));
             throw e.getCause();
           }
         }
         else {
+          LOG.error(ExceptionUtils.getStackTrace(e));
           throw e;
         }
       } catch (InvocationTargetException e) {
@@ -127,11 +129,13 @@ public class RetryingHMSHandler implemen
           caughtException = e.getCause();
         }
         else {
+          LOG.error(ExceptionUtils.getStackTrace(e.getCause()));
           throw e.getCause();
         }
       }
 
       if (retryCount >= retryLimit) {
+        LOG.error(ExceptionUtils.getStackTrace(caughtException));
         // Since returning exceptions with a nested "cause" can be a problem in
         // Thrift, we are stuffing the stack trace into the message itself.
         throw new MetaException(ExceptionUtils.getStackTrace(caughtException));