Skip to content

Commit

Permalink
Adding order by
Browse files Browse the repository at this point in the history
Reverted planner logic to align with DuckDB
  • Loading branch information
Dtenwolde committed Jul 4, 2023
1 parent 895d805 commit 7d68352
Showing 1 changed file with 1 addition and 20 deletions.
21 changes: 1 addition & 20 deletions src/planner/planner.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,6 @@ void Planner::CreatePlan(SQLStatement &statement) {
this->types = {LogicalTypeId::UNKNOWN};
this->plan = nullptr;
parameters_resolved = false;
} catch (const BinderException &ex) {
throw;
} catch (const Exception &ex) {
auto &config = DBConfig::GetConfig(context);

Expand All @@ -68,28 +66,11 @@ void Planner::CreatePlan(SQLStatement &statement) {
break;
}
}

if (!this->plan) {
throw;
}
} catch (std::exception &ex) {
auto &config = DBConfig::GetConfig(context);

this->plan = nullptr;
for (auto &extension_op : config.operator_extensions) {
auto bound_statement =
extension_op->Bind(context, *this->binder, extension_op->operator_info.get(), statement);
if (bound_statement.plan != nullptr) {
this->names = bound_statement.names;
this->types = bound_statement.types;
this->plan = std::move(bound_statement.plan);
break;
}
}

if (!this->plan) {
throw;
}
throw;
}
this->properties = binder->properties;
this->properties.parameter_count = parameter_count;
Expand Down

0 comments on commit 7d68352

Please sign in to comment.