Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: introduce executor v2 for the new planner #725

Merged
merged 3 commits into from
Nov 4, 2022
Merged

Conversation

wangrunji0408
Copy link
Member

This PR starts migrating executors to the new planner. The new version is put into a new module executor_v2. This PR finishes the migration of scan, projection, filter, and explain executor. Now the new planner is able to work from end to end!

> \v2
switched to planner v2
in 0.000s
> select * from nation where N_NATIONKEY = 24;
+----+---------------+---+----------------------------------------------------------------------------------------------------------------+
| 24 | UNITED STATES | 1 | y final packages. slow foxes cajole quickly. quickly silent platelets breach ironic accounts. unusual pinto be |
+----+---------------+---+----------------------------------------------------------------------------------------------------------------+
in 0.067s
> explain select * from nation where N_NATIONKEY = 24;
+--------------------------------------------------+
| Projection: [$0.0, $0.1, $0.2, $0.3] (cost=6000) |
|   Filter: ($0.0 = 24) (cost=5100)                |
|     Scan: [$0.0, $0.1, $0.2, $0.3] (cost=4000)   |
|                                                  |
+--------------------------------------------------+
in 0.036s

Signed-off-by: Runji Wang <wangrunji0408@163.com>
Signed-off-by: Runji Wang <wangrunji0408@163.com>
Signed-off-by: Runji Wang <wangrunji0408@163.com>
@wangrunji0408 wangrunji0408 merged commit de6dd81 into main Nov 4, 2022
@wangrunji0408 wangrunji0408 deleted the wrj/executor_v2 branch November 4, 2022 02:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants