----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16339/#review30694 -----------------------------------------------------------
common/src/java/org/apache/hadoop/hive/conf/HiveConf.java <https://reviews.apache.org/r/16339/#comment58798> This should be on by default. After HIVE-5297 storing column values in non-canonical forms should not happen. ql/src/test/queries/clientpositive/alter_partition_coltype.q <https://reviews.apache.org/r/16339/#comment58799> This should work even with config set to true. No? - Ashutosh Chauhan On Dec. 19, 2013, 2:17 a.m., Sergey Shelukhin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16339/ > ----------------------------------------------------------- > > (Updated Dec. 19, 2013, 2:17 a.m.) > > > Review request for hive and Ashutosh Chauhan. > > > Repository: hive-git > > > Description > ------- > > see JIRA > > > Diffs > ----- > > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java fa3e048 > metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreDirectSql.java > a98d9d1 > metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java > 04d399f > > metastore/src/java/org/apache/hadoop/hive/metastore/parser/ExpressionTree.java > 93e9942 > ql/src/test/queries/clientpositive/alter_partition_coltype.q 5479afb > ql/src/test/queries/clientpositive/annotate_stats_part.q 83510e3 > ql/src/test/queries/clientpositive/dynamic_partition_skip_default.q 397a220 > ql/src/test/results/clientpositive/alter_partition_coltype.q.out 27b1fbc > ql/src/test/results/clientpositive/annotate_stats_part.q.out 87fb980 > ql/src/test/results/clientpositive/dynamic_partition_skip_default.q.out > baee525 > > Diff: https://reviews.apache.org/r/16339/diff/ > > > Testing > ------- > > > Thanks, > > Sergey Shelukhin > >
