sunchao commented on PR #5567: URL: https://github.com/apache/datafusion-comet/pull/5567#issuecomment-5506662725
@viirya Agreed: this is a pre-existing `mapsort` defect, and #5567 adds another path that exposes it. The existing map-grouping rule is independent of the nested-hash option. Thanks for isolating the fix in #5630; I’m reviewing that PR now. Once this branch incorporates it, I’ll recheck the OFFSET → map-repartition coverage here, including the native-operator assertions. -- 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]
