Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13901 )
Change subject: Add default value to switch in kudu-util ...................................................................... Add default value to switch in kudu-util KuduDataTypeToColumnType in kudu-util.cc has a switch that translates kudu::client::KuduColumnSchema::DataType to PrimitiveType. It returns PrimitiveType::INVALID_TYPE if an invalid type is passed, but it was done after the switch instead of the 'default' case which leads to a warning if a new unhandled Kudu type is added to the header. Change-Id: I81a8ab4b912d443582c8ac3899f370a4b838e934 Reviewed-on: http://gerrit.cloudera.org:8080/13901 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/exec/kudu-util.cc 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/13901 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I81a8ab4b912d443582c8ac3899f370a4b838e934 Gerrit-Change-Number: 13901 Gerrit-PatchSet: 3 Gerrit-Owner: Attila Bukor <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Anonymous Coward (314) Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
