sunchao commented on PR #6437: URL: https://github.com/apache/datafusion-comet/pull/6437#issuecomment-5982693819
Rebased in `25d3a5a896dd9072eba1fd14726e678e53472583`. Rebased the Union change onto the rebased #6037 prerequisite. Kept main's TopK support, operators-crate imports and release of never-consumed Arrow streams while preserving Union-owned cleanup. Native core/test compilation passes with the same validation-only public DataFusion companion (`c9a142d834b6b47c2ac15524171bbf84f3022f73`), and full Spark 4.1 reactor test compilation, Rust formatting, Spotless, suite registration and whitespace checks pass. Dependency pins and the lockfile remain unchanged. The existing draft blocker remains: Comet's released DataFusion55.1 dependency does not contain the prepared-build APIs, even though upstream #25491 is merged. Runtime tests/benchmarks were not rerun on this rebased head, and ordinary native CI cannot pass until that dependency prerequisite is available. -- 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]
