thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-30 11:32:03 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix review comments

Browse Source
This commit is contained in:
Anton Popov 2022-04-28 12:52:01 +00:00
parent cc29bb74ca
commit be70d237b2
1 changed files with 3 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/DataTypes/ObjectUtils.cpp
View File

@ -713,18 +713,11 @@ Field FieldVisitorReplaceScalars::operator()(const Array & x) const
size_t FieldVisitorToNumberOfDimensions::operator()(const Array & x) const size_t FieldVisitorToNumberOfDimensions::operator()(const Array & x) const
{ {
const size_t size = x.size(); const size_t size = x.size();
std::optional<size_t> dimensions; size_t dimensions = 0;
for (size_t i = 0; i < size; ++i) for (size_t i = 0; i < size; ++i)
{ dimensions = std::max(dimensions, applyVisitor(*this, x[i]));
size_t current_dimensions = applyVisitor(*this, x[i]);
if (!dimensions)
dimensions = current_dimensions;
else
dimensions = std::max(*dimensions, current_dimensions);
}
return 1 + dimensions.value_or(0); return 1 + dimensions;
} }
} }