[DRILL-6832] Remove old "unmanaged" sort implementation

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.14.0
• Fix Version/s: 1.18.0
• Labels: ready-to-commit
• Assignee: Paul Rogers
• Reporter: Paul Rogers
• Reviewer: ihuzenko#1
• Created: 06/Nov/18 21:55
• Updated: 02/Jan/20 16:45
• Resolved: 02/Jan/20 16:45

Description

Several releases back Drill introduced a new "managed" external sort that enhanced the sort operator's memory management. To be safe, at the time, the new version was controlled by an option, with the ability to revert to the old version.

The new version has proven to be stable. The time has come to remove the old version.

• Remove the implementation in physical.impl.xsort.
• Move the implementation from physical.impl.xsort.managed to the parent package.
• Remove the conditional code in the batch creator.
• Remove the option that allowed disabling the new version.

Issue Links

• links to: GitHub Pull Request #1929