You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@hive.apache.org by pengcheng xiong <px...@hortonworks.com> on 2015/04/30 06:39:31 UTC

Review Request 33714: genFileSinkPlan should use the generated SEL's RR for the partition col of FS

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33714/
-----------------------------------------------------------

Review request for hive, Ashutosh Chauhan and John Pullokkaran.


Repository: hive-git


Description
-------

Right now, when Hive writes data into a bucketed table, it will use the last OP to generate RS-SEL and then FS. However, the context rsCtx carries 
partition column from the last OP rather than from the SEL, which makes FS use the wrong partition column.


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 7f6a180 

Diff: https://reviews.apache.org/r/33714/diff/


Testing
-------


Thanks,

pengcheng xiong