-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[Enhancement] Optimize performance of arrays_overlaping #49668
[Enhancement] Optimize performance of arrays_overlaping #49668
Conversation
02d0b7b
to
a9de696
Compare
a9de696
to
8b79bbe
Compare
dd0e868
to
2057ed3
Compare
@Mergifyio rebase main |
✅ Branch has been successfully rebased |
f8b65c0
to
92a80e3
Compare
@Mergifyio rebase main |
✅ Branch has been successfully rebased |
92a80e3
to
adf29b7
Compare
https://github.com/Mergifyio rebase main |
adf29b7
to
2a869d3
Compare
✅ Branch has been successfully rebased |
https://github.com/Mergifyio rebase main |
2a869d3
to
217d784
Compare
✅ Branch has been successfully rebased |
https://github.com/Mergifyio rebase main |
217d784
to
eaf4343
Compare
✅ Branch has been successfully rebased |
https://github.com/Mergifyio rebase main |
Signed-off-by: trueeyu <[email protected]>
Signed-off-by: trueeyu <[email protected]>
[FE Incremental Coverage Report]✅ pass : 0 / 0 (0%) |
[BE Incremental Coverage Report]✅ pass : 126 / 133 (94.74%) file detail
|
https://github.com/Mergifyio backport branch-3.3 |
✅ Backports have been created
|
Signed-off-by: trueeyu <[email protected]> (cherry picked from commit 20d8075)
Signed-off-by: trueeyu <[email protected]> (cherry picked from commit 20d8075)
) (#51539) Signed-off-by: trueeyu <[email protected]> Co-authored-by: trueeyu <[email protected]>
https://github.com/Mergifyio backport branch-3.2 |
✅ Backports have been created
|
Signed-off-by: trueeyu <[email protected]> (cherry picked from commit 20d8075)
) (#51557) Signed-off-by: trueeyu <[email protected]> Co-authored-by: trueeyu <[email protected]>
https://github.com/Mergifyio backport branch-3.1 |
✅ Backports have been created
|
Signed-off-by: trueeyu <[email protected]> (cherry picked from commit 20d8075) # Conflicts: # be/src/exprs/array_functions.tpp
) (#51561) Signed-off-by: trueeyu <[email protected]> Co-authored-by: trueeyu <[email protected]>
) Signed-off-by: trueeyu <[email protected]> Signed-off-by: zhiminr.ren <[email protected]>
Why I'm doing:
Datum
is not an efficient data structure, andarrrasy_overlapping
heavily relies on Datum for processing, resulting in poor performance.The second parameter of
arrrasy_overlapping
is usually const, but there is currently no special handling.What I'm doing:
PerformaceTest: 27.76s -> 2.74s
Before optimization: 27.76s
After optimization: 2.74s
What type of PR is this:
Does this PR entail a change in behavior?
If yes, please specify the type of change:
Checklist:
Bugfix cherry-pick branch check:
Documentation PRs only:
If you are submitting a PR that adds or changes English documentation and have not
included Chinese documentation, then you can check the box to request GPT to translate the
English doc to Chinese. Please ensure to uncheck the Do not translate box if translation is needed.
The workflow will generate a new PR with the Chinese translation after this PR is merged.