You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
Related to fix for #10165. We have a couple of specific handling for executor side broadcasts where we add in an extra shuffle to go from GpuExchange -> GpuColumnarToRow -> Exchange (EXECUTOR_BROADCAST). We do this because the BroadcastHashJoin is expecting the Exchange node there and not a ColumnarToRow exec. This works but its likely adding in extra overhead.
Perhaps we can make a special GPUExchange that has a doExecute and handles the columnar to row itself, but we should analyze the performance impact and if something like that will work.
Is your feature request related to a problem? Please describe.
Related to fix for #10165. We have a couple of specific handling for executor side broadcasts where we add in an extra shuffle to go from GpuExchange -> GpuColumnarToRow -> Exchange (EXECUTOR_BROADCAST). We do this because the BroadcastHashJoin is expecting the Exchange node there and not a ColumnarToRow exec. This works but its likely adding in extra overhead.
Perhaps we can make a special GPUExchange that has a doExecute and handles the columnar to row itself, but we should analyze the performance impact and if something like that will work.
after fix for #10165 The plan looks like:
You can see that we have an extra shuffle in there.
The text was updated successfully, but these errors were encountered: