wForget commented on issue #12474:
URL: https://github.com/apache/gluten/issues/12474#issuecomment-4987875787

   > [@wForget](https://github.com/wForget) 
[@philo-he](https://github.com/philo-he) Could you approve these 
workflows([#12523](https://github.com/apache/gluten/pull/12523))? I solved this 
problem with the help of AI, so I'm not fully sure whether it passes all the 
unit tests.
   
   @littlelittlewhite09 Thank you for your contribution! I fixed the issue in 
https://github.com/apache/gluten/pull/12514, and the Velox CI has passed 
successfully. Would you be willing to take over the fix by cherry-picking the 
commits or copying the changes to your branch?
   
   The ClickHouse backend may have the same issue, so I made the fix in 
`gluten-substrait`. However, two unit tests are currently failing, possibly 
because the execution plans have changed. cc @baibaichen @zzcclp
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to