impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Marcel Kornacker (Code Review)" <>
Subject [Impala-CR](cdh5-trunk) IMPALA-1740: Add support for skip.header.line.count.
Date Thu, 07 Apr 2016 03:32:46 GMT
Marcel Kornacker has posted comments on this change.

Change subject: IMPALA-1740: Add support for skip.header.line.count.

Patch Set 17:

File be/src/exec/

Line 646:     if (scan_node_->skip_header_line_count() > 1) {
would be good to have a test case somewhere against a multi-format table (w/ text and parquet)
where skip.header.linecount is set
File fe/src/main/java/com/cloudera/impala/analysis/

Line 88:       if (error.length() > 0) {
single line?
File fe/src/main/java/com/cloudera/impala/analysis/

Line 245:     HdfsFileFormat fileFormat = HdfsFileFormat.fromThrift(fileFormat_);
make this block a static function this class or the AlterTable.. class and then call that
from both places.
File fe/src/main/java/com/cloudera/impala/catalog/

Line 1232:             "Could not parse table property %s: %s (expecting integer value >=
i think you can return the same error message as in line 1237 here
File fe/src/main/java/com/cloudera/impala/planner/

Line 120:   private int skipHeaderLineCount_;
then init to 0?
File fe/src/main/java/com/cloudera/impala/planner/

Line 1290:       if (hdfsTable.hasSkipHeaderLineCount()) {
the planner shouldn't have to deal with this, do it in the scan node c'tor (and pass in the
tbl ref)

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I595f01a165d41499ca1956fe748ba3840a6eb543
Gerrit-PatchSet: 17
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Lars Volker <>
Gerrit-Reviewer: Lars Volker <>
Gerrit-Reviewer: Marcel Kornacker <>
Gerrit-Reviewer: Skye Wanderman-Milne <>
Gerrit-HasComments: Yes

View raw message