----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13862/#review25819 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/optimizer/correlation/ReduceSinkDeDuplication.java <https://reviews.apache.org/r/13862/#comment50366> In here. if(result[0] <= 0) throw new SemanticException("Sort columns and order don't match. Try hive.optimize.reducesinkdeduplication=false;"); Another sanity check. - Ashutosh Chauhan On Aug. 30, 2013, 3:29 p.m., Yin Huai wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13862/ > ----------------------------------------------------------- > > (Updated Aug. 30, 2013, 3:29 p.m.) > > > Review request for hive. > > > Bugs: HIVE-5149 > https://issues.apache.org/jira/browse/HIVE-5149 > > > Repository: hive-git > > > Description > ------- > > https://mail-archives.apache.org/mod_mbox/hive-user/201308.mbox/%3CCAG6Lhyex5XPwszpihKqkPRpzri2k=m4qgc+cpar5yvr8sjt...@mail.gmail.com%3E > > > Diffs > ----- > > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/correlation/ReduceSinkDeDuplication.java > c380a2d > ql/src/test/results/clientpositive/groupby2_map_skew.q.out da7a128 > ql/src/test/results/clientpositive/groupby_cube1.q.out a52f4eb > ql/src/test/results/clientpositive/groupby_rollup1.q.out f120471 > ql/src/test/results/clientpositive/reduce_deduplicate_extended.q.out > 3297ebb > > Diff: https://reviews.apache.org/r/13862/diff/ > > > Testing > ------- > > > Thanks, > > Yin Huai > >