> On Sept. 12, 2014, 6:58 p.m., Gunther Hagleitner wrote: > > ql/src/test/queries/clientpositive/alter_partition_change_col.q, line 7 > > <https://reviews.apache.org/r/25320/diff/1/?file=676166#file676166line7> > > > > it'd be good to test: > > > > - dynamic partition case (no value for one partition specs, multiple > > partition specs). does this work? If it does, what happens if there are > > some partitions that cannot be changed? (some partitions already have a > > column x others don't) > > - tables with multiple partitions > > - reordering columns > > - null/default partition > > - negative cases (name clash, column doesn't exist, etc)
- Dynamic partition spec does not work for alter table statement, looks like the syntax only supports specifying one partition at a time. - I've updated alter_partition_change_col.q to have multiple partitions - Reordering the columns in a partition does not look very effective since it appears that the partition column names are ignored when reading data from the table. The 1st column from the partition is treated as if it is the 1st column of the table, and so on. - Are null/default partitions possible? Tried to test this but I didn't get it working - Adding negative test cases - Jason ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25320/#review53197 ----------------------------------------------------------- On Sept. 3, 2014, 11:57 p.m., Jason Dere wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25320/ > ----------------------------------------------------------- > > (Updated Sept. 3, 2014, 11:57 p.m.) > > > Review request for hive and Ashutosh Chauhan. > > > Bugs: HIVE-7971 > https://issues.apache.org/jira/browse/HIVE-7971 > > > Repository: hive-git > > > Description > ------- > > Allow change/replace/add column to work on partitions > > > Diffs > ----- > > ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java e42bbdd > ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java > 05cde3e > ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g 25cd3a5 > ql/src/java/org/apache/hadoop/hive/ql/plan/AlterTableDesc.java 8517319 > ql/src/test/queries/clientpositive/alter_partition_change_col.q > PRE-CREATION > ql/src/test/results/clientpositive/alter_partition_change_col.q.out > PRE-CREATION > > Diff: https://reviews.apache.org/r/25320/diff/ > > > Testing > ------- > > New qfile test added > > > Thanks, > > Jason Dere > >
