andygrove commented on PR #25853: URL: https://github.com/apache/datafusion/pull/25853#issuecomment-6560189742
Merged main again and addressed the latest reviews: - Main added its own Case H to `aggregate_memory_spill.slt`, so this PR's case is now Case I. - Dictionary-typed state (@jayzhan211): c5cc3af12, details in the review thread. It sizes and compacts dictionaries whose values are mostly unique, and leaves the others shared as on main. - `string_agg` case added, and @alamb's questions answered inline. @comphead, the dictionary handling is new since your approval, in case you want to look at c5cc3af12. -- 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]
