wForget opened a new pull request, #10241:
URL: https://github.com/apache/incubator-gluten/pull/10241

   Backport #9770 to branch-1.4
   
   ## What changes were proposed in this pull request?
   
   Bug fix for velox + celeborn + columnar shuffle.
   
   gluten changes the shuffle writer type of celeborn in 
https://github.com/apache/incubator-gluten/blob/main/backends-velox/src/main/scala/org/apache/gluten/backendsapi/velox/VeloxSparkPlanExecApi.scala#L555-L557
   
   As a result, we cannot determine whether the shuffle writer in 
VeloxCelebornColumnarShuffleWriter is sort or not,  can only  use the passed in 
shuffle time metric .
   
   (Fixes: \#9769)
   
   ## How was this patch tested?
   
   manual tests
   
   


-- 
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