uros-db commented on PR #46597: URL: https://github.com/apache/spark/pull/46597#issuecomment-2119214355
As for the modified open hash map, that does sound promising - I suppose changing `update` in order to inject collationKey should be enough? it seems that `merge` should then work by default but then of course there's the problem of preserving one of the actual values - you correctly noticed that we can't just return collationKey, as that might not be present in the original array I suppose a separate map might do the trick here (mapping collationKey to original string value), and since we don't have preference towards which value gets returned, simply returning the first one that appeared is considered correct behaviour -- 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]
