danielwei816 commented on issue #7393: URL: https://github.com/apache/datafusion-comet/issues/7393#issuecomment-6590215334
Thanks for looking into this. I understand the concern about maintaining and testing compatibility with vendor forks. Would you consider a focused fix using the existing `makeCopy()` approach from #207? That PR addressed a similar `FileSourceScanExec` signature mismatch on Databricks. This looks like another call site where preserving the runtime's existing fields could avoid the hard-coded `copy()` signature. I haven't tested that patch yet, but I can validate it against the public EMR image and on EMR Serverless, and share the reproducer and results. If the change also passes the upstream Spark tests, would you be open to accepting it while keeping the existing support policy? -- 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]
