-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26481/
-----------------------------------------------------------
(Updated Oct. 17, 2014, 2:43 a.m.)
Review request for hive, Gunther Hagleitner and Thejas Nair.
Changes
-------
changes based on review feedback:
- Added check if dynamic partitioning is enabled
- Only fail for 0 partitions if fully specified partition spec
- Changed alter_partition_change_col.q to have multiple partition columns
- added negative test to test when dynamic partitioning disabled
Bugs: HIVE-8411
https://issues.apache.org/jira/browse/HIVE-8411
Repository: hive-git
Description
-------
Allow partial partition spec for some alter table statements, such as:
alter table alter_partition_change_col1 partition (p1) change column c2 c2
decimal(10,0);
Diffs (updated)
-----
ql/src/java/org/apache/hadoop/hive/ql/ErrorMsg.java 31978fe
ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java 3a6eb39
ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java 5b8f301
ql/src/java/org/apache/hadoop/hive/ql/plan/AlterTableDesc.java 298bbca
ql/src/test/queries/clientnegative/alter_partition_partial_spec_dyndisabled.q
PRE-CREATION
ql/src/test/queries/clientpositive/alter_partition_change_col.q baabb9f
ql/src/test/results/clientnegative/alter_partition_partial_spec_dyndisabled.q.out
PRE-CREATION
ql/src/test/results/clientpositive/alter_partition_change_col.q.out 7123e40
Diff: https://reviews.apache.org/r/26481/diff/
Testing
-------
updated qfile test
Thanks,
Jason Dere
