TomaYoko commented on PR #28501: URL: https://github.com/apache/doris/pull/28501#issuecomment-1859392980
> @TomaYoko And please add comment on multi_map to explain why we need multimap here, so that other developers will notice that when do some refactor work. @yiguolei What I have tried before is that when string type such as CHAR/VARCHAR/STRING type(other type are not this phenomenon) used as original column in a minus/except statement, same column will use the same ColRef id which will cause the order_map only saves only one col ，thus ,will cause the result is incorrect. this similar pr is mentioned in #28440, which I have discussed with @zhiqiang-hhhh -- 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]
