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.

Sign up for GitHub

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

Jump to bottom

[WIP]executor: inline projection for join #14517

Closed
wants to merge 10 commits into from
Closed

[WIP]executor: inline projection for join #14517

wants to merge 10 commits into from

Conversation

SunRunAway
Copy link
Contributor

@SunRunAway SunRunAway commented Jan 17, 2020 •
edited
Loading

What problem does this PR solve?

part of #14428

What is changed and how it works?

Check List

Tests

  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)

Code changes

  • Has exported function/method change

Side effects

  • Increased code complexity

Related changes

  • None

Release note

  • Support inline projection for join operator.

@SunRunAway SunRunAway added the sig/execution SIG execution label Jan 17, 2020
@SunRunAway SunRunAway requested a review from a team as a code owner January 17, 2020 09:18
@ghost ghost requested review from eurekaka and lzmhhh123 and removed request for a team January 17, 2020 09:18
@SunRunAway SunRunAway added the sig/planner SIG: Planner label Jan 17, 2020
@SunRunAway SunRunAway requested review from a team and removed request for eurekaka and lzmhhh123 January 17, 2020 09:18
@ghost ghost requested review from francis0407 and alivxxx and removed request for a team January 17, 2020 09:18
@SunRunAway SunRunAway changed the title executor: inline projection for join [WIP]executor: inline projection for join Jan 17, 2020
@SunRunAway

This comment has been minimized.

Sign in to view
1 similar comment
@SunRunAway

This comment has been minimized.

Sign in to view
@alivxxx alivxxx removed their request for review January 19, 2020 03:30
@SunRunAway SunRunAway force-pushed the inlineprojection-hashjoin3 branch from 2323e30 to 2cb4900 Compare January 19, 2020 03:59
@SunRunAway SunRunAway requested a review from a team as a code owner January 19, 2020 04:04
@ghost ghost requested review from qw4990 and XuHuaiyu and removed request for a team January 19, 2020 04:04
@SunRunAway SunRunAway force-pushed the inlineprojection-hashjoin3 branch from 5474bf7 to 56382b8 Compare January 19, 2020 09:51
@SunRunAway SunRunAway force-pushed the inlineprojection-hashjoin3 branch from 1809de9 to 719a52e Compare January 19, 2020 10:52
@SunRunAway
Copy link
Contributor Author

/run-all-tests

@SunRunAway SunRunAway force-pushed the inlineprojection-hashjoin3 branch from 719a52e to 6b1a1f6 Compare January 19, 2020 13:14
@SunRunAway SunRunAway force-pushed the inlineprojection-hashjoin3 branch from 6b1a1f6 to 1c821da Compare February 5, 2020 07:46
@SunRunAway SunRunAway force-pushed the inlineprojection-hashjoin3 branch from 1c821da to 37b1719 Compare February 5, 2020 07:48
@SunRunAway SunRunAway mentioned this pull request Feb 5, 2020
Open
9 tasks
@SunRunAway SunRunAway closed this Feb 7, 2020
@SunRunAway
Copy link
Contributor Author

I'll file anthor PR to seperate it into executor part and planner part.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
sig/execution SIG execution sig/planner SIG: Planner type/new-feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@SunRunAway