andygrove commented on PR #1793: URL: https://github.com/apache/datafusion-comet/pull/1793#issuecomment-2916340347
Thanks @rluvaton but this PR does not appear to help with the CI issue (the tests are still failing - see https://github.com/apache/datafusion-comet/actions/runs/15278587551/job/42973846520?pr=1793). The scans in Comet are pretty confusing. In the native plan we either have DataFusion's `DataSourceExec`, or a Comet `ScanExec` that reads data from the JVM. The source of data for `ScanExec` could be a broadcast or shuffle exchange, or a Parquet scan either using `native_comet` or `native_iceberg_compat`. When using `native_comet` there is buffer re-use and that is when we need to use Comet's `FilterExec`. -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
