Review Board 1.7.22


Allow merge join after order-bys

Review Request #5280 - Created May 29, 2012 and submitted

Dmitriy Ryaboy
PIG-2673
Reviewers
ashutoshc, daijy, julien
pig
Merge joins should be allowed to work after order-by operators.
In this patch, we relax the merge join conditions to allow such an operator (and fix up some logic bugs that previously existed in the checks).
Note that it's technically possible with this patch to sort by one field, and merge-join by another, as we don't do a deep field-level inspection. 
We intend to add that level of safety at a later date.
Unit tests + manual run on sample data.
Review request changed
Updated (June 27, 2012, 10:07 a.m.)
  • changed from pending to submitted