Author: cws
Date: Tue Aug 23 05:36:17 2011
New Revision: 1160551

URL: http://svn.apache.org/viewvc?rev=1160551&view=rev
Log:
HIVE-2338. Alter table always throws an unhelpful error on failure (Sohan Jain 
via cws)

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

Modified: 
hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java
URL: 
http://svn.apache.org/viewvc/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java?rev=1160551&r1=1160550&r2=1160551&view=diff
==============================================================================
--- 
hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java
 (original)
+++ 
hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java
 Tue Aug 23 05:36:17 2011
@@ -193,7 +193,6 @@ public class HiveAlterHandler implements
     } finally {
       if (!success) {
         msdb.rollbackTransaction();
-        throw new MetaException("Committing the alter table transaction was 
not successful.");
       }
       if (success && moveData) {
         // change the file name in hdfs
@@ -220,6 +219,9 @@ public class HiveAlterHandler implements
         }
       }
     }
+    if (!success) {
+      throw new MetaException("Committing the alter table transaction was not 
successful.");
+    }
   }
 
   private boolean checkPartialPartKeysEqual(List<FieldSchema> oldPartKeys,


Reply via email to