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

Enumerator & mapper improvements #580

Closed
2 of 3 tasks
andyfengHKU opened this issue May 4, 2022 · 1 comment
Closed
2 of 3 tasks

Enumerator & mapper improvements #580

andyfengHKU opened this issue May 4, 2022 · 1 comment
Assignees

Comments

@andyfengHKU
Copy link
Contributor

andyfengHKU commented May 4, 2022

  • Rename enumerator to planner except for join order enumerator
  • Keep schema as planner local concept. Avoid giving schema to mapper. A possible solution is, instead of giving actual factorzation group position to logical operator. We give a logical position and mapper can map from logical position to a physical position.
  • Split mapper to different cpp file per operator.
@andyfengHKU andyfengHKU self-assigned this May 4, 2022
@andyfengHKU
Copy link
Contributor Author

For the second task, we choose a different approach. Now we are giving schema to operator so dependency becomes

physical operator -[depends on]-> schema -[depends on]-> logical operator. We can move logical operator freely as long as we recompute schema after moving. See PR #1119 and #1121

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

No branches or pull requests

1 participant