61yao commented on PR #9968: URL: https://github.com/apache/pinot/pull/9968#issuecomment-1350696846
> let's don't remove this for now until we have a configuration for hash function pluggability, the code is not used but the logic is still valid. what do you think? Yeah. I didn't remove the underlying singleton exchange. I just update the exchange when join key is empty to hash exchange and remove the supported exchange type. We should be able to add it back fast by just adding the exchange type to mailbox. -- 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]
