rkrishn7 commented on PR #17632: URL: https://github.com/apache/datafusion/pull/17632#issuecomment-3308372036
> I'm going to rename the `Hash` UDF to `RepartitionHash` and (1) make it clear that this is an internal hash function subject to change (as suggested by @Dandandan) and (2) leave it as AHash for now to punt on selecting a hash function compatible with distributed hash calculations. If it's solely for internal use at the moment I wonder if it's better to just make it a private `PhysicalExpr` and keep it within the hash join module -- 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]
