----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50054/#review142442 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java <https://reviews.apache.org/r/50054/#comment208006> Yes, it is necessary. For example, TS0 and TS1 union together and there is a FS2. TezCompiler will generate TS0-FS2 and also TS1-FS2(cloned). We generate a single statsTask for FS2, but we still need to gather stats for FS2(cloned). ql/src/java/org/apache/hadoop/hive/ql/parse/GenTezUtils.java (line 308) <https://reviews.apache.org/r/50054/#comment208007> Yes, it is a typo. - pengcheng xiong On July 14, 2016, 11:18 p.m., pengcheng xiong wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50054/ > ----------------------------------------------------------- > > (Updated July 14, 2016, 11:18 p.m.) > > > Review request for hive and Ashutosh Chauhan. > > > Repository: hive-git > > > Description > ------- > > HIVE-14236 > > > Diffs > ----- > > itests/src/test/resources/testconfiguration.properties 73fcb03 > ql/src/java/org/apache/hadoop/hive/ql/exec/FileSinkOperator.java 812a943 > ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java c626add > ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java f555741 > ql/src/java/org/apache/hadoop/hive/ql/parse/GenTezUtils.java 32bf24d > ql/src/java/org/apache/hadoop/hive/ql/plan/FileSinkDesc.java 07ed4fd > ql/src/java/org/apache/hadoop/hive/ql/stats/StatsCollectionContext.java > ae6f2ac > ql/src/java/org/apache/hadoop/hive/ql/stats/fs/FSStatsPublisher.java > 3a49b30 > ql/src/test/queries/clientpositive/union_stats.q PRE-CREATION > ql/src/test/results/clientpositive/tez/union_stats.q.out PRE-CREATION > ql/src/test/results/clientpositive/union_stats.q.out PRE-CREATION > > Diff: https://reviews.apache.org/r/50054/diff/ > > > Testing > ------- > > > Thanks, > > pengcheng xiong > >
