Marton Greber has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/23409 )
Change subject: [client] fix ClangTidy warning in schema.cc
......................................................................
[client] fix ClangTidy warning in schema.cc
This is to address the following ClangTidy's warning:
src/kudu/client/schema.cc:173:5: warning: switch has 3 consecutive identical
branches [bugprone-branch-clone]
case kudu::DECIMAL32: return KuduColumnSchema::DECIMAL;
^
This changelist doesn't contain any functional modifications.
Change-Id: I11745a942d7d4c9c2711fcb279e13dec197ce582
Reviewed-on: http://gerrit.cloudera.org:8080/23409
Reviewed-by: Marton Greber <[email protected]>
Tested-by: Marton Greber <[email protected]>
Reviewed-by: Gabriella Lotz <[email protected]>
---
M src/kudu/client/schema.cc
1 file changed, 4 insertions(+), 3 deletions(-)
Approvals:
Marton Greber: Looks good to me, approved; Verified
Gabriella Lotz: Looks good to me, but someone else must approve
--
To view, visit http://gerrit.cloudera.org:8080/23409
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I11745a942d7d4c9c2711fcb279e13dec197ce582
Gerrit-Change-Number: 23409
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Gabriella Lotz <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>