[IMPALA-5922] Scanners should include file and offset information in errors

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: Impala 2.11.0
• Component/s: Backend
• Epic Color: ghx-label-4
• Assignee: Unassigned
• Reporter: Lars Volker
• Created: 12/Sep/17 16:43
• Updated: 28/Mar/19 18:22

Description

Currently we have to print the location of a parse error in BaseSequenceScanner::GetNextInternal():

state_->LogError(ErrorMsg(TErrorCode::SEQUENCE_SCANNER_PARSE_ERROR, 
    stream_->filename(), stream_->file_offset(),                    
    (stream_->eof() ? "(EOF)" : "")));                              

Instead, the scanners should include this information when constructing the error, which will allow us to simplify the error handling in the base class.

Issue Links

• relates to: IMPALA-5890 Segmentation fault in ScannerContext::Stream::GetBytesInternal(long, unsigned char**, bool, long*)