[DRILL-4174] fix for DRILL-4081 mistakenly regresses the fix for DRILL-3786

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.4.0
• Fix Version/s: 1.5.0
• Component/s: Execution - Relational Operators
• Assignee: Abdel Hakim Deneche
• Reporter: Abdel Hakim Deneche
• Reviewer: Abhishek Girish
• Created: 09/Dec/15 01:53
• Updated: 20/Sep/16 18:24
• Resolved: 09/Jan/16 13:35

Description

As part of fixing DRILL-3786, ExternalSortBatch counts how many batches are held in memory in a local variable "totalBatches" to make sure sort spills to disk before we have too many batches for the SelectionVector4.
The fix for DRILL-4081 removed the line that incremented totalBatches which causes the previous fix to no longer work