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

[ML-224] Optimize table merge performance issues #225

Merged
merged 224 commits into from
Apr 13, 2023

Conversation

minmingzhu
Copy link
Collaborator

@minmingzhu minmingzhu commented Jul 25, 2022

What changes were proposed in this pull request?

  1. Resolved the problem of wasted memory , because useless data didn't free immediately.
  2. Resolved the problem of poor merge performance

closes #224

Does this PR also require the following changes?

  • CI
  • Documentation
  • Example

No.

minmingzhu and others added 30 commits April 7, 2022 17:41
2. modify compiler config
3. add unit test

Signed-off-by: minmingzhu <[email protected]>
Signed-off-by: minmingzhu <[email protected]>
Signed-off-by: minmingzhu <[email protected]>
Signed-off-by: minmingzhu <[email protected]>
Signed-off-by: minmingzhu <[email protected]>
2. remove CI test becaue homogentable only support GPU

Signed-off-by: minmingzhu <[email protected]>
Signed-off-by: minmingzhu <[email protected]>
Signed-off-by: minmingzhu <[email protected]>
Signed-off-by: minmingzhu <[email protected]>
Signed-off-by: minmingzhu <[email protected]>
Signed-off-by: minmingzhu <[email protected]>
Signed-off-by: minmingzhu <[email protected]>
Signed-off-by: minmingzhu <[email protected]>
Signed-off-by: minmingzhu <[email protected]>
Signed-off-by: minmingzhu <[email protected]>
@minmingzhu minmingzhu changed the base branch from oneapi_interface to master April 8, 2023 17:51
Signed-off-by: minmingzhu <[email protected]>
@minmingzhu minmingzhu changed the title [ML-224]Resolved table merge [ML-224] Resolved table merge Apr 12, 2023
@minmingzhu minmingzhu changed the title [ML-224] Resolved table merge [ML-224] Optimize table merge performance issues Apr 13, 2023
@xwu99 xwu99 merged commit 2f6b35c into oap-project:master Apr 13, 2023
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.

[Core] Did not erase the shared_ptr elements in Vector. Lead to wasted memory.
3 participants