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

Merge

Browse Source
This commit is contained in:
Evgeniy Gatov 2014-05-26 08:48:33 +04:00
commit d6ad0254da
18 changed files with 314 additions and 90 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
dbms/include/DB/AggregateFunctions/AggregateFunctionArray.h
View File

@ -59,7 +59,7 @@ public:
nested_func->create(place);
}
void destroy(AggregateDataPtr place) const
void destroy(AggregateDataPtr place) const noexcept
{
nested_func->destroy(place);
}

2
dbms/include/DB/AggregateFunctions/AggregateFunctionIf.h
View File

@ -57,7 +57,7 @@ public:
nested_func->create(place);
}
void destroy(AggregateDataPtr place) const
void destroy(AggregateDataPtr place) const noexcept
{
nested_func->destroy(place);
}

2
dbms/include/DB/AggregateFunctions/AggregateFunctionMerge.h
View File

@ -59,7 +59,7 @@ public:
nested_func->create(place);
}
void destroy(AggregateDataPtr place) const
void destroy(AggregateDataPtr place) const noexcept
{
nested_func->destroy(place);
}

2
dbms/include/DB/AggregateFunctions/AggregateFunctionState.h
View File

@ -55,7 +55,7 @@ public:
nested_func->create(place);
}
void destroy(AggregateDataPtr place) const
void destroy(AggregateDataPtr place) const noexcept
{
nested_func->destroy(place);
}

4
dbms/include/DB/AggregateFunctions/IAggregateFunction.h
View File

@ -59,7 +59,7 @@ public:
virtual void create(AggregateDataPtr place) const = 0;
/// Уничтожить данные для агрегации.
virtual void destroy(AggregateDataPtr place) const = 0;
virtual void destroy(AggregateDataPtr place) const noexcept = 0;
/// Уничтожать данные не обязательно.
virtual bool hasTrivialDestructor() const = 0;
@ -118,7 +118,7 @@ public:
new (place) Data;
}
void destroy(AggregateDataPtr place) const
void destroy(AggregateDataPtr place) const noexcept
{
data(place).~Data();
}

2
dbms/include/DB/Columns/ColumnArray.h
View File

@ -280,7 +280,7 @@ public:
void getPermutation(bool reverse, size_t limit, Permutation & res) const
{
size_t s = size();
if (limit > s)
if (limit >= s)
limit = 0;
res.resize(s);

5
dbms/include/DB/Columns/ColumnFixedString.h
View File

@ -122,7 +122,8 @@ public:
bool operator()(size_t lhs, size_t rhs) const
{
/// TODO: memcmp тормозит.
return positive == (0 > memcmp(&parent.chars[lhs * parent.n], &parent.chars[rhs * parent.n], parent.n));
int res = memcmp(&parent.chars[lhs * parent.n], &parent.chars[rhs * parent.n], parent.n);
return positive ? (res < 0) : (res > 0);
}
};
@ -133,7 +134,7 @@ public:
for (size_t i = 0; i < s; ++i)
res[i] = i;
if (limit > s)
if (limit >= s)
limit = 0;
if (limit)

16
dbms/include/DB/Columns/ColumnString.h
View File

@ -266,9 +266,11 @@ public:
less(const ColumnString & parent_) : parent(parent_) {}
bool operator()(size_t lhs, size_t rhs) const
{
return positive == (0 > strcmp(
int res = strcmp(
reinterpret_cast<const char *>(&parent.chars[parent.offsetAt(lhs)]),
reinterpret_cast<const char *>(&parent.chars[parent.offsetAt(rhs)])));
reinterpret_cast<const char *>(&parent.chars[parent.offsetAt(rhs)]));
return positive ? (res < 0) : (res > 0);
}
};
@ -279,7 +281,7 @@ public:
for (size_t i = 0; i < s; ++i)
res[i] = i;
if (limit > s)
if (limit >= s)
limit = 0;
if (limit)
@ -308,9 +310,11 @@ public:
bool operator()(size_t lhs, size_t rhs) const
{
return positive == (0 > collator.compare(
int res = collator.compare(
reinterpret_cast<const char *>(&parent.chars[parent.offsetAt(lhs)]), parent.sizeAt(lhs),
reinterpret_cast<const char *>(&parent.chars[parent.offsetAt(rhs)]), parent.sizeAt(rhs)));
reinterpret_cast<const char *>(&parent.chars[parent.offsetAt(rhs)]), parent.sizeAt(rhs));
return positive ? (res < 0) : (res > 0);
}
};
@ -322,7 +326,7 @@ public:
for (size_t i = 0; i < s; ++i)
res[i] = i;
if (limit > s)
if (limit >= s)
limit = 0;
if (limit)

2
dbms/include/DB/Columns/ColumnTuple.h
View File

@ -182,7 +182,7 @@ public:
for (size_t i = 0; i < rows; ++i)
res[i] = i;
if (limit > rows)
if (limit >= rows)
limit = 0;
if (limit)

2
dbms/include/DB/Columns/ColumnVector.h
View File

@ -149,7 +149,7 @@ public:
for (size_t i = 0; i < s; ++i)
res[i] = i;
if (limit > s)
if (limit >= s)
limit = 0;
if (limit)

2
dbms/include/DB/Interpreters/Aggregator.h
View File

@ -470,7 +470,7 @@ protected:
Names key_names;
AggregateDescriptions aggregates;
std::vector<IAggregateFunction *> aggregate_functions;
std::vector<bool> is_final;
std::vector<char> is_final;
size_t keys_size;
size_t aggregates_size;
/// Нужно ли класть в AggregatedDataVariants::without_key агрегаты для ключей, не попавших в max_rows_to_group_by.

33
dbms/src/Interpreters/Aggregator.cpp
View File

@ -265,14 +265,15 @@ void Aggregator::mergeDataImpl(
if (!inserted)
{
for (size_t i = 0; i < aggregates_size; ++i)
{
aggregate_functions[i]->merge(
Method::getAggregateData(res_it->second) + offsets_of_aggregate_states[i],
Method::getAggregateData(it->second) + offsets_of_aggregate_states[i]);
for (size_t i = 0; i < aggregates_size; ++i)
aggregate_functions[i]->destroy(
Method::getAggregateData(it->second) + offsets_of_aggregate_states[i]);
}
Method::getAggregateData(it->second) = nullptr;
}
else
{
@ -331,8 +332,6 @@ void Aggregator::destroyImpl(
for (typename Method::const_iterator it = method.data.begin(); it != method.data.end(); ++it)
{
for (size_t i = 0; i < aggregates_size; ++i)
/// Если аггрегатная функция не может быть финализирована, то за ее удаление отвечает ColumnAggregateFunction
if (is_final[i])
{
char * data = Method::getAggregateData(it->second);
@ -528,6 +527,8 @@ Block Aggregator::convertToBlock(AggregatedDataVariants & data_variants, bool fi
key_columns[i]->reserve(rows);
}
try
{
for (size_t i = 0; i < aggregates_size; ++i)
{
is_final[i] = final && aggregate_functions[i]->canBeFinal();
@ -587,6 +588,22 @@ Block Aggregator::convertToBlock(AggregatedDataVariants & data_variants, bool fi
final_aggregate_columns, data_variants.key_sizes, start_row);
else if (data_variants.type != AggregatedDataVariants::WITHOUT_KEY)
throw Exception("Unknown aggregated data variant.", ErrorCodes::UNKNOWN_AGGREGATED_DATA_VARIANT);
}
catch (...)
{
/** Работа с состояниями агрегатных функций недостаточно exception-safe.
* Если часть столбцов aggregate_columns была resize-на, но значения не были вставлены,
* то эти столбцы будут в некорректном состоянии
* (ColumnAggregateFunction попытаются в деструкторе вызвать деструкторы у элементов, которых нет),
* а также деструкторы будут вызываться у AggregatedDataVariants.
* Поэтому, вручную "откатываем" их.
*/
for (size_t i = 0; i < aggregates_size; ++i)
if (aggregate_columns[i])
aggregate_columns[i]->clear();
throw;
}
if (!final)
{
@ -650,10 +667,12 @@ AggregatedDataVariantsPtr Aggregator::merge(ManyAggregatedDataVariants & data_va
AggregatedDataWithoutKey & current_data = current.without_key;
for (size_t i = 0; i < aggregates_size; ++i)
{
aggregate_functions[i]->merge(res_data + offsets_of_aggregate_states[i], current_data + offsets_of_aggregate_states[i]);
for (size_t i = 0; i < aggregates_size; ++i)
aggregate_functions[i]->destroy(current_data + offsets_of_aggregate_states[i]);
}
current_data = nullptr;
}
if (res->type == AggregatedDataVariants::KEY_64)
@ -775,8 +794,6 @@ void Aggregator::destroyAllAggregateStates(AggregatedDataVariants & result)
AggregatedDataWithoutKey & res_data = result.without_key;
for (size_t i = 0; i < aggregates_size; ++i)
/// Если аггрегатная функция не может быть финализирована, то за ее удаление отвечает ColumnAggregateFunction
if (is_final[i])
aggregate_functions[i]->destroy(res_data + offsets_of_aggregate_states[i]);
}

2
dbms/src/Interpreters/sortBlock.cpp
View File

@ -97,7 +97,7 @@ void sortBlock(Block & block, const SortDescription & description, size_t limit)
for (size_t i = 0; i < size; ++i)
perm[i] = i;
if (limit > size)
if (limit >= size)
limit = 0;
bool need_collation = false;

2
dbms/src/Storages/StorageFactory.cpp
View File

@ -193,7 +193,7 @@ StoragePtr StorageFactory::get(
* - (для Replicated) Путь к таблице в ZooKeeper
* - (для Replicated) Имя реплики в ZooKeeper
* - имя столбца с датой;
* - (не обязательно) имя столбца для семплирования (запрос с SAMPLE x будет выбирать строки, у которых в этом столбце значение меньше, чем x*UINT32_MAX);
* - (не обязательно) выражение для семплирования (запрос с SAMPLE x будет выбирать строки, у которых в этом столбце значение меньше, чем x*UINT32_MAX);
* - выражение для сортировки (либо скалярное выражение, либо tuple из нескольких);
* - index_granularity;
* - (для Collapsing) имя столбца, содержащего тип строчки с изменением "визита" (принимающего значения 1 и -1).

100
dbms/tests/queries/0_stateless/00044_sorting_by_string_descending.reference Normal file
View File

@ -0,0 +1,100 @@
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc

1
dbms/tests/queries/0_stateless/00044_sorting_by_string_descending.sql Normal file
View File

@ -0,0 +1 @@
SELECT s FROM (SELECT materialize('abc') AS s FROM system.numbers LIMIT 100) ORDER BY s DESC

100
dbms/tests/queries/0_stateless/00045_sorting_by_fixed_string_descending.reference Normal file
View File

@ -0,0 +1,100 @@
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc
abc

1
dbms/tests/queries/0_stateless/00045_sorting_by_fixed_string_descending.sql Normal file
View File

@ -0,0 +1 @@
SELECT s FROM (SELECT toFixedString(materialize('abc'), 3) AS s FROM system.numbers LIMIT 100) ORDER BY s DESC