search

apache / incubator-gluten

Gluten is a middle layer responsible for offloading JVM-based SQL engines' execution to native engines.
https://gluten.apache.org/
Apache License 2.0
1.22k stars 437 forks source link

[VL] Branch 1.2: Backport fixes for #7243 #7943

Closed zhztheplayer closed 2 days ago

zhztheplayer commented 1 week ago

Backport fixes for https://github.com/apache/incubator-gluten/issues/7243.

The work is partly based on previous efforts https://github.com/apache/incubator-gluten/pull/7921 and https://github.com/apache/incubator-gluten/pull/7648.

Fixes for https://github.com/apache/incubator-gluten/issues/6736 are not included, and https://github.com/apache/incubator-gluten/pull/7165 is ported as a temporary solution.

github-actions[bot] commented 1 week ago

Run Gluten Clickhouse CI on x86

zhztheplayer commented 1 week ago

cc @wForget @zsmj2017

zhztheplayer commented 1 week ago

@wForget @zsmj2017

Would you want to help verify this since CI is passing now, thanks.

zsmj2017 commented 1 week ago

@wForget @zsmj2017

Would you want to help verify this since CI is passing now, Thanks.

I am testing some spill hang etls and will give response in 2024/11/15

wForget commented 1 week ago

@wForget @zsmj2017

Would you want to help verify this since CI is passing now, thanks.

Thank you, I will verify this later.

zsmj2017 commented 1 week ago

@wForget @zsmj2017 Would you want to help verify this since CI is passing now, Thanks.

I am testing some spill hang etls and will give response in 2024/11/15

@zhztheplayer Based on the tests, the previously stable spill-hang job no longer hangs.（5/5 jobs passed)

zhztheplayer commented 3 days ago

Guys, are we good to merge this? Thanks.