kosiew commented on code in PR #19285: URL: https://github.com/apache/datafusion/pull/19285#discussion_r2618156963
########## datafusion/physical-plan/src/aggregates/mod.rs: ########## @@ -73,6 +75,39 @@ mod row_hash; mod topk; mod topk_stream; +/// Returns true if TopK aggregation data structures support the provided key and value types. +/// +/// This function checks whether both the key type (used for grouping) and value type +/// (used in min/max aggregation) can be handled by the TopK aggregation heap and hash table. +/// Supported types include Arrow primitives (integers, floats, decimals, intervals) and +/// UTF-8 strings (`Utf8`, `LargeUtf8`, `Utf8View`). +/// +/// # Example Review Comment: I'll remove the example. -- 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]
