Re: [PR] feat: add CopyExec and move CopyExec handling to Spark [datafusion-comet]

2025-07-14 Thread via GitHub
dharanad commented on PR #2001: URL: https://github.com/apache/datafusion-comet/pull/2001#issuecomment-3070721634 > Yep, I think this is headed in the right direction. Really anywhere you see us having hard-coded rules about inserting CopyExec in planner.rs we'd like to have consolidated d

Re: [PR] feat: add CopyExec and move CopyExec handling to Spark [datafusion-comet]

2025-07-14 Thread via GitHub
mbutrovich commented on PR #2001: URL: https://github.com/apache/datafusion-comet/pull/2001#issuecomment-3069581934 Yep, I think this is headed in the right direction. Really anywhere you see us having hard-coded rules about inserting CopyExec in planner.rs we'd like to have consolidated d

Re: [PR] feat: add CopyExec and move CopyExec handling to Spark [datafusion-comet]

2025-07-12 Thread via GitHub
dharanad commented on PR #2001: URL: https://github.com/apache/datafusion-comet/pull/2001#issuecomment-3065953457 @mbutrovich I would appreciate it if you could take a look at this PR and provide some early feedback on whether I'm headed in the right direction -- This is an automated mes

[PR] feat: add CopyExec and move CopyExec handling to Spark [datafusion-comet]

2025-07-08 Thread via GitHub
dharanad opened a new pull request, #2001: URL: https://github.com/apache/datafusion-comet/pull/2001 ## Which issue does this PR close? Closes #1995 ## Rationale for this change ## What changes are included in this PR? ## How are these chang