[DRILL-5512] Standardize error handling in ScanBatch

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.10.0
• Fix Version/s: 1.11.0
• Labels: ready-to-commit
• Assignee: Paul Rogers
• Reporter: Paul Rogers
• Reviewer: Sorabh Hamirwasia
• Created: 15/May/17 21:34
• Updated: 03/Jun/17 04:53
• Resolved: 03/Jun/17 04:47

Description

ScanBatch is the Drill operator executor that handles most readers. Like most Drill operators, it uses an ad-hoc set of error detection and reporting methods that evolved over Drill development.

This ticket asks to standardize on error handling as outlined in DRILL-5083. This basically means reporting all errors as a UserException rather than using the IterOutcome.STOP return status or using the FragmentContext.fail() method.

This work requires the new error codes introduced in DRILL-5511, and is a step toward making readers aware of vector size limits.

Issue Links

• is part of: DRILL-5211 Queries fail due to direct memory fragmentation
• links to: GitHub Pull Request #838