spark-rapids icon indicating copy to clipboard operation
spark-rapids copied to clipboard

[BUG] GpuInMemoryTableScanExec doesn't wrap it's children to GPU versions

Open razajafri opened this issue 1 year ago • 3 comments

Describe the bug The GpuInMemoryTableScanExec doesn't convert it's children to GPU which causes tests to fail if the InMemoryTableScanExec has children. We have been getting away with this for quite sometime

razajafri avatar Mar 27 '24 17:03 razajafri

Can you give more details on the test failures that necessitate this change?

mattahrens avatar Apr 02 '24 20:04 mattahrens

We are not testing for this. I saw this while working on adding tests for #10518. I can create a test for this and add to this issue.

razajafri avatar Apr 03 '24 15:04 razajafri

Here is the code I am referring to, we can see that we are passing the parameters without converting them to the GPU versions.

I think this is not OK but I have to write a test for it before I can prove it.

razajafri avatar Apr 03 '24 15:04 razajafri