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

[GLUTEN-7243][VL] A follow-up fix for #7748 #7935

Closed zhztheplayer closed 1 week ago

zhztheplayer commented 1 week ago

7748 missed a local object name to make RAII work with correct scope.

Without the change, the suspend state will be cleared immediately since the local object will be destroyed right after it's created. So hasNext() call will not consider the current driver as suspended. As a result #7748 will not work and spill may still cause hanging.

The patch fixes the issue.

github-actions[bot] commented 1 week ago

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