Skip to content

Conversation

@GlutenPerfBot
Copy link
Contributor

Upstream Velox's New Commits:

79f71a611 by Bikramjeet Vig, fix(Peeling): Support peeling of dictionaries without nulls wrapping a constant (13194)
2691af870 by Jialiang Tan, fix: Make ArbitrationParticipant::reclaim not throw (13226)
087fc6944 by Amit Dutta, misc: Throw user error when unsupported character found during casting to number types (13227)
1455df114 by Xiaoxuan Meng, feat: Introduce task barrier (13087)
ca0d20811 by Jialiang Tan, fix: Fix shrink capacity log on abort allowed signal (13228)

Upstream Velox's New Commits:
79f71a6 by Bikramjeet Vig, fix(Peeling): Support peeling of dictionaries without nulls wrapping a constant (#13194)
2691af8 by Jialiang Tan, fix: Make ArbitrationParticipant::reclaim not throw (#13226)
087fc69 by Amit Dutta, misc: Throw user error when unsupported character found during casting to number types (#13227)
1455df1 by Xiaoxuan Meng, feat: Introduce task barrier (#13087)
ca0d208 by Jialiang Tan, fix: Fix shrink capacity log on abort allowed signal (#13228)

Signed-off-by: GlutenPerfBot <[email protected]>
@github-actions
Copy link

github-actions bot commented May 5, 2025

#6887

@marin-ma
Copy link
Contributor

marin-ma commented May 6, 2025

@JkSelf There are some merge conflicts with smj commit oap-project/velox@b327046#diff-70e6a4169952565c158043e3b944394e488220364692fad35fa3397566c8a082

Could you help to take a look? Thanks!

@JkSelf
Copy link
Contributor

JkSelf commented May 6, 2025

@JkSelf There are some merge conflicts with smj commit oap-project/velox@b327046#diff-70e6a4169952565c158043e3b944394e488220364692fad35fa3397566c8a082

Could you help to take a look? Thanks!

@marin-ma Sure. I will take a look. Thanks.

@JkSelf
Copy link
Contributor

JkSelf commented May 6, 2025

@marin-ma I have fixed the conflicts in facebookincubator/velox#11771. Can you have a try?

@zhouyuan
Copy link
Contributor

zhouyuan commented May 6, 2025

@JkSelf Thanks! I made some quick fix on the patch

@zhouyuan zhouyuan merged commit 9a58c2f into apache:main May 6, 2025
46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants