Hello Thomas Tauber-Marshall, Anonymous Coward (314), Adar Dembo, Grant Henke,
Csaba Ringhofer,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/13901
to review the following change.
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
---
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/01/13901/1
--
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: newchange
Gerrit-Change-Id: I81a8ab4b912d443582c8ac3899f370a4b838e934
Gerrit-Change-Number: 13901
Gerrit-PatchSet: 1
Gerrit-Owner: Attila Bukor <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Anonymous Coward (314)
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]>
