thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-27 01:51:59 +00:00
Code Issues Packages Projects Releases Wiki Activity

clickhouse: leaving the smallest column instead of the first [#CONV-2807].

Browse Source
This commit is contained in:
Michael Kolupaev 2013-06-10 14:24:40 +00:00
parent 078f598ea0
commit 8a738aee11
1 changed files with 24 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
dbms/src/Interpreters/ExpressionActions.cpp
View File

@ -371,6 +371,25 @@ void ExpressionActions::execute(Block & block) const
}
}
static std::string getAnyColumn(const NamesAndTypesList & columns)
{
NamesAndTypesList::const_iterator it = columns.begin();
size_t min_size = it->second->isNumeric() ? it->second->getSizeOfField() : 100;
String res = it->first;
for (; it != columns.end(); ++it)
{
size_t current_size = it->second->isNumeric() ? it->second->getSizeOfField() : 100;
if (current_size < min_size)
{
min_size = current_size;
res = it->first;
}
}
return res;
}
void ExpressionActions::finalize(const Names & output_columns)
{
typedef std::set<std::string> NameSet;
@ -385,6 +404,10 @@ void ExpressionActions::finalize(const Names & output_columns)
final_columns.insert(name);
}
/// Не будем блок пустым, чтобы не потерять количество строк в нем.
if (final_columns.empty())
final_columns.insert(getAnyColumn(input_columns));
NameSet used_columns = final_columns;
for (size_t i = 0; i < actions.size(); ++i)
@ -413,8 +436,7 @@ void ExpressionActions::finalize(const Names & output_columns)
for (int i = static_cast<int>(sample_block.columns()) - 1; i >= 0; --i)
{
const std::string & name = sample_block.getByPosition(i).name;
/// Не удаляем последний столбец.
if (!final_columns.count(name) && sample_block.columns() > 1)
if (!final_columns.count(name))
add(Action::removeColumn(name));
}
}