parthchandra commented on PR #6422: URL: https://github.com/apache/datafusion-comet/pull/6422#issuecomment-5919290089
The diff is a faithful `-x` cherry-pick of #5421 with byte-identical fix logic, branch-1.1 already carries it, and CI is green. One thing to fix before merge: The "What changes are included" section of the description still says the `RevertNativeForTransitionHeavyStages` hunk is dropped. That's now stale - the follow-up commit `61c366d9368` adds both the guard call and the full `hasUnsafeMixedAggregateAtStageBoundary` method to that file (around `RevertNativeForTransitionHeavyStages.scala:117`, guard call site ~line 95). Since the description is the permanent record a future auditor reads, please rewrite it to say the guard is ported to match main, not dropped. -- 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]
