You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by cw...@apache.org on 2011/08/23 07:36:18 UTC
svn commit: r1160551 -
/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java
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,