Skip to content

feat(hash_join): implement basic Inner/Left/Right/Full Join operations#43

Merged
KKould merged 7 commits intoKipData:mainfrom
KKould:main
Aug 11, 2023
Merged

feat(hash_join): implement basic Inner/Left/Right/Full Join operations#43
KKould merged 7 commits intoKipData:mainfrom
KKould:main

Conversation

@KKould
Copy link
Copy Markdown
Member

@KKould KKould commented Aug 4, 2023

found a problem that needs to be fixed:

  • when the Projection operator executes eval_column, the reading data is abnormal due to the duplication of the ColumnIdx subscript of the Catalog So Binder::bind_project need to support join

What problem does this PR solve?

Add corresponding issue link with summary if exists -->

Issue link:

What is changed and how it works?

Code changes

  • Has Rust code change
  • Has CI related scripts change

Check List

Tests

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

Side effects

  • Performance regression: Consumes more CPU
  • Performance regression: Consumes more Memory
  • Breaking backward compatibility

Note for reviewer

found a problem that needs to be fixed:
- when the `Projection` operator executes eval_column, the reading data is abnormal due to the duplication of the ColumnIdx subscript of the Catalog
So `Binder::bind_project` need to support join
@KKould KKould added the enhancement New feature or request label Aug 4, 2023
@KKould KKould added this to the v0.1 milestone Aug 4, 2023
@KKould KKould self-assigned this Aug 4, 2023
@KKould KKould merged commit cec22f2 into KipData:main Aug 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant