Repository: hive Updated Branches: refs/heads/master 9dfdc2ffe -> 5919e9026
HIVE-14118: Make the alter partition exception more meaningful (Aihua Xu, reviewed by Sergio Peña) Project: http://git-wip-us.apache.org/repos/asf/hive/repo Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/5919e902 Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/5919e902 Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/5919e902 Branch: refs/heads/master Commit: 5919e90267a7568440a63101ff06368f0b0b1379 Parents: 9dfdc2f Author: Aihua Xu <aihu...@apache.org> Authored: Fri Jul 1 14:45:38 2016 -0400 Committer: Aihua Xu <aihu...@apache.org> Committed: Fri Jul 1 14:45:38 2016 -0400 ---------------------------------------------------------------------- .../org/apache/hadoop/hive/metastore/HiveAlterHandler.java | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hive/blob/5919e902/metastore/src/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java ---------------------------------------------------------------------- diff --git a/metastore/src/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java b/metastore/src/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java index 68c6e44..7b84595 100644 --- a/metastore/src/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java +++ b/metastore/src/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java @@ -519,11 +519,8 @@ public class HiveAlterHandler implements AlterHandler { updatePartColumnStats(msdb, dbname, name, oldTmpPart.getValues(), tmpPart); } msdb.alterPartitions(dbname, name, partValsList, new_parts); - } catch (InvalidObjectException e) { - throw new InvalidOperationException("alter is not possible"); - } catch (NoSuchObjectException e){ - //old partition does not exist - throw new InvalidOperationException("alter is not possible"); + } catch (InvalidObjectException | NoSuchObjectException e) { + throw new InvalidOperationException("Alter partition operation fails: " + e); } return oldParts; }