----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38871/#review101246 -----------------------------------------------------------
Ship it! Ship It! - Prasanth_J On Sept. 30, 2015, 1:10 a.m., Ashutosh Chauhan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38871/ > ----------------------------------------------------------- > > (Updated Sept. 30, 2015, 1:10 a.m.) > > > Review request for hive and Prasanth_J. > > > Bugs: HIVE-11972 > https://issues.apache.org/jira/browse/HIVE-11972 > > > Repository: hive-git > > > Description > ------- > > [Refactor] Improve determination of dynamic partitioning columns in FileSink > Operator > > > Diffs > ----- > > ql/src/java/org/apache/hadoop/hive/ql/exec/FileSinkOperator.java 2604d5d > ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java bcf85a4 > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/ConstantPropagateProcFactory.java > 5c6a6df > ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java 02fbdfe > ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java dda28b0 > ql/src/java/org/apache/hadoop/hive/ql/plan/DynamicPartitionCtx.java 24db7d0 > ql/src/test/org/apache/hadoop/hive/ql/exec/TestFileSinkOperator.java > c6ae030 > > Diff: https://reviews.apache.org/r/38871/diff/ > > > Testing > ------- > > Existing regression suite. > > > Thanks, > > Ashutosh Chauhan > >