This is an automated email from the ASF dual-hosted git repository. alexey pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/kudu.git
commit dd4fcccf5a7c6a0ede70ab77f79df6277f6c0115 Author: Alexey Serbin <[email protected]> AuthorDate: Tue Sep 9 22:21:02 2025 -0700 [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]> --- src/kudu/client/schema.cc | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/kudu/client/schema.cc b/src/kudu/client/schema.cc index d5831000a..27e150c66 100644 --- a/src/kudu/client/schema.cc +++ b/src/kudu/client/schema.cc @@ -169,9 +169,10 @@ KuduColumnSchema::DataType FromInternalDataType(kudu::DataType type) { case kudu::STRING: return KuduColumnSchema::STRING; case kudu::BINARY: return KuduColumnSchema::BINARY; case kudu::BOOL: return KuduColumnSchema::BOOL; - case kudu::DECIMAL32: return KuduColumnSchema::DECIMAL; - case kudu::DECIMAL64: return KuduColumnSchema::DECIMAL; - case kudu::DECIMAL128: return KuduColumnSchema::DECIMAL; + case kudu::DECIMAL32: + case kudu::DECIMAL64: + case kudu::DECIMAL128: + return KuduColumnSchema::DECIMAL; default: LOG(FATAL) << "Unexpected internal data type: " << type; } }
