Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9283
Change subject: Add KuduDataTypeToColumnType default case ...................................................................... Add KuduDataTypeToColumnType default case Given the Impala build breaks when a non-exhaustive enum switch statement is used, there is no way the KuduDataTypeToColumnType switch statement could fall through to return `ColumnType(PrimitiveType::INVALID_TYPE);` and still compile. Instead, this should be moved to the default case in the switch statment. Because Kudu is adding a new Decimal type, this will prevent a compilation failure if/when the Kudu version is upgraded as well. Change-Id: I8f7d9e16070b66274107a2bff52cbe8f3ae9b8f1 --- M be/src/exec/kudu-util.cc 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/83/9283/1 -- To view, visit http://gerrit.cloudera.org:8080/9283 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I8f7d9e16070b66274107a2bff52cbe8f3ae9b8f1 Gerrit-Change-Number: 9283 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke <granthe...@gmail.com> Gerrit-Reviewer: Thomas Tauber-Marshall <tmarsh...@cloudera.com>
