Rachelint commented on PR #11718: URL: https://github.com/apache/datafusion/pull/11718#issuecomment-2260956150
Duckdb has the similar check(but just use u16 prefix of hash): https://github.com/duckdb/duckdb/blob/f92559f42c118075baaa8daafc437954eb5c85ec/src/execution/aggregate_hashtable.cpp#L379-L386 And mentioned by @alamb , the related pr in duckdb is interesting to see https://github.com/duckdb/duckdb/pull/9575 -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
