tursom/memgraph
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Add template to cost estimator

Browse Source
This commit is contained in:
Josip Mrden
2023-06-23 13:33:18 +02:00
parent 2f335f2507
commit 80a0d12e08
1 changed files with 4 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/query/plan/cost_estimator.hpp
View File

@@ -296,8 +296,10 @@ class CostEstimator : public HierarchicalLogicalOperatorVisitor {
}
bool PreVisit(Apply &op) override {
// Get the cost of the main branch
op.input_->Accept(*this);
// Estimate cost on the subquery branch independently, use a copy
auto last_scope = scopes_.back();
double subquery_cost = EstimateCostOnBranch(&op.subquery_, last_scope);
@@ -385,14 +387,8 @@ class CostEstimator : public HierarchicalLogicalOperatorVisitor {
return scope.symbol_stats[symbol.name()];
}
void SaveStatsFor(const Symbol &symbol, storage::LabelIndexStats index_stats) {
scopes_.back().symbol_stats[symbol.name()] = SymbolStatistics{
.cardinality = index_stats.count,
.degree = index_stats.avg_degree,
};
}
void SaveStatsFor(const Symbol &symbol, storage::LabelPropertyIndexStats index_stats) {
template <typename T>
void SaveStatsFor(const Symbol &symbol, T index_stats) {
scopes_.back().symbol_stats[symbol.name()] = SymbolStatistics{
.cardinality = index_stats.count,
.degree = index_stats.avg_degree,