----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67800/#review205649 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/optimizer/FixedBucketPruningOptimizer.java Lines 90 (patched) <https://reviews.apache.org/r/67800/#comment288536> This needs the pruned partition list loop (to make sure all partitions have the same bucket-count). This is probably something we can prevent from happening in Bucketing version = 2 so that tables with new bucketing version will never allow an ALTER TABLE clustered by (id) buckets from changing a partition's bucket count. - Gopal V On July 3, 2018, 12:23 a.m., Deepak Jaiswal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67800/ > ----------------------------------------------------------- > > (Updated July 3, 2018, 12:23 a.m.) > > > Review request for hive and Gopal V. > > > Bugs: HIVE-20039 > https://issues.apache.org/jira/browse/HIVE-20039 > > > Repository: hive-git > > > Description > ------- > > Bucket pruning: Left Outer Join on bucketed table gives wrong result. > The context was reused by all the predicates. Instead use TS op directly. > > > Diffs > ----- > > > data/files/bucket_pruning/l3_clarity__l3_monthly_dw_factplan_datajoin_1_s2_2018022300104_1/000000_0 > PRE-CREATION > > data/files/bucket_pruning/l3_clarity__l3_monthly_dw_factplan_dw_stg_2018022300104_1/000000_0 > PRE-CREATION > data/files/bucket_pruning/l3_clarity__l3_snap_number_2018022300104/000000_0 > PRE-CREATION > data/files/bucket_pruning/l3_monthly_dw_dimplan/000056_0 PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/FixedBucketPruningOptimizer.java > 2debacacb5 > ql/src/test/queries/clientpositive/tez_fixed_bucket_pruning.q PRE-CREATION > ql/src/test/results/clientpositive/llap/tez_fixed_bucket_pruning.q.out > PRE-CREATION > > > Diff: https://reviews.apache.org/r/67800/diff/1/ > > > Testing > ------- > > > Thanks, > > Deepak Jaiswal > >
