[DRILL-7583] Remove STOP status in favor of fail-fast

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.17.0
• Fix Version/s: 1.18.0
• Labels: ready-to-commit
• Assignee: Paul Rogers
• Reporter: Paul Rogers
• Reviewer: Igor Guzenko
• Created: 13/Feb/20 18:16
• Updated: 21/Feb/20 15:49
• Resolved: 21/Feb/20 15:49

Description

The original error solution was a complex process of a) setting a failed flag, b) telling all upstream operators they have failed, c) returning a STOP status. Drill has long supported a "fail-fast" error path based on throwing an exception; relying on the fragment executor to clean up the operator stack. Recent revisions have converted most operators to use the simpler fail-fast strategy based on throwing an exception instead of using the older STOP approach. This change simply removes the old, now-unused STOP based path.

Issue Links

• links to: GitHub Pull Request #1981