mbutrovich commented on issue #4859: URL: https://github.com/apache/datafusion-comet/issues/4859#issuecomment-5205496628
> @mixermt if we can compile comet with custom DF, I would happy to compile comet on our side and perform heavy testing on real data. > @adriangb It'd be great to test this in depth to make sure we don't ship a large DataFusion change that benefits no one. I pushed the Comet-side fix to the `df55` branch, along with the DataFusion bump past apache/datafusion#24090. The DataFusion pin in that branch's `native/Cargo.toml` already points at the fixed commit directly, so no custom patching is needed on your end, a normal `cargo build -p datafusion-comet` from `native/` against `df55` picks it up. CI is still running on the PR (apache/datafusion-comet#5262). I'd rather wait until it's green before asking anyone to build against it, so I'll comment back here once it passes. At that point pulling `df55` and testing against real data would be a great way to validate the leaf pruning end to end, thanks for offering. -- 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]
