[IMPALA-5217] KuduTableSink checks null constraints incorrectly

• Type: Bug
• Status: Resolved
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: Impala 2.8.0
• Fix Version/s: Impala 2.9.0
• Component/s: Backend
• Labels: kudu
• Target Version: Impala 2.9.0
• Epic Color: ghx-label-5
• Assignee: Thomas Tauber-Marshall
• Reporter: Thomas Tauber-Marshall
• Created: 17/Apr/17 21:35
• Updated: 21/Apr/17 14:28
• Resolved: 21/Apr/17 14:28

Description

Something I noticed while looking at kudu-table-sink.cc: we use the referenced_columns map to translate between the index into the output exprs 'j' and the index into columns in the Kudu table 'col', but we incorrectly use 'j' when calling into the Kudu table schema to check the nullability of columns.