forked from cmu-db/noisepage
-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Description
TODOs:
- At
TrafficCop::BindQuery:- Implement the binder logic at :
BindNodeVisitor::Visit(parser::AlterTableStatement)
- Implement the binder logic at :
- At
TrafficCop::OptimizeBoundQuery:- Implement operator trasnformer at :
QueryToOperatorTransformer::Visit(AlterTableStatement) - Implement Logical Operator for Alter at :
logical_operator.h - Implement Logical to Physical rules at: 'rule.cpp' and 'implementation_rules.cpp'
- To generate a physical plan, implement physical operator at :
physical_operator.h' forAlterTable` - Implement plan node for AlterTable at :
alter_table_plan_node.h?
Maybe Should we break the AlterTable based on different commands? Like different AlterCmd should have different (sub) plan node? Needs to keep in mind there are tons of commands possible.
- Implement PlanGenerator::Visit for
AlterTablephysical plan operator atplan_generator.cpp
- Implement operator trasnformer at :
- At
TrafficCop::ExecutePortal:- TrafficCop to execute the physical plan for AlteTable Query, so implement maybe
ExecuteAlterStatementatpostgres_network_commands.cpp:ExecutePortal - Implement the executors at
DDLExecutors,AlterTableExectutor
- TrafficCop to execute the physical plan for AlteTable Query, so implement maybe
Implement Add column first through the entire pipeline first. And then refactor from there.
Change column type might have to scan through the table, should be delayed.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels