[DRILL-3345] TestWindowFrame fails to properly check cases involving multiple batches

• Type: Bug
• Status: Closed
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: 1.1.0
• Fix Version/s: 1.1.0
• Component/s: Execution - Relational Operators
• Labels: window_function
• Assignee: Abdel Hakim Deneche
• Reporter: Abdel Hakim Deneche
• Created: 23/Jun/15 20:21
• Updated: 25/Jun/15 14:35
• Resolved: 25/Jun/15 14:35

Description

as part of adding unit tests to TestWindowFrame, I added a debug option to MSorter to force the batches passed downstream to be of a specific size. This was supposed to help test edge cases involving multiple data batches while controlling precisely when a partition and/or window frame ends in a batch.

Turns out the change to MSorter was incomplete and all cases end up processing one single big batch of data.

The purpose of this JIRA issue is to fix this, add some check to detect when the batches are no longer split correctly and make sure all unit tests are passing.

Issue Links

• is depended upon by: DRILL-3327 row_number function returns incorrect result when only order by clause is specified