mirrors/scylladb
1
0
Fork 0
mirror of https://github.com/scylladb/scylladb.git synced 2026-04-23 01:50:35 +00:00
Code Issues Packages Projects Releases Wiki Activity

cql3: statement_restrictions: multi-key clustering restrictions one layer deeper

Browse Source 
For the multi column binary operator case, perform more of the work at
prepare time in preparation for consolidating the analysis.
This commit is contained in:
Avi Kivity
2024-11-13 22:59:43 +02:00
parent b520e74128
commit c60e3d5cf7
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
cql3/restrictions/statement_restrictions.cc
View File

@@ -2718,13 +2718,14 @@ statement_restrictions::build_get_clustering_bounds_fn() const {
};
}
if (find_binop(_clustering_prefix_restrictions[0].filter, is_multi_column)) { // FIXME: adjust for solve_for
return [&] (const query_options& options) -> std::vector<query::clustering_range> {
bool all_natural = true, all_reverse = true; ///< Whether column types are reversed or natural.
for (auto& r : _clustering_prefix_restrictions | std::views::transform(&predicate::filter)) { // TODO: move to constructor, do only once.
using namespace expr;
const auto& binop = expr::as<binary_operator>(r);
if (is_clustering_order(binop)) {
return [this] (const query_options& options) -> std::vector<query::clustering_range> {
return {range_from_raw_bounds(_clustering_prefix_restrictions, options, *_schema)};
};
}
for (auto& element : expr::as<tuple_constructor>(binop.lhs).elements) {
auto& cv = expr::as<column_value>(element);
@@ -2735,6 +2736,7 @@ statement_restrictions::build_get_clustering_bounds_fn() const {
}
}
}
return [this, all_natural, all_reverse] (const query_options& options) -> std::vector<query::clustering_range> {
return get_multi_column_clustering_bounds(options, _schema, _clustering_prefix_restrictions,
all_natural, all_reverse);
};