xiedeyantu commented on PR #4706: URL: https://github.com/apache/calcite/pull/4706#issuecomment-3701116107
> The problem is that there is no spec about the meaning of a constructor call of a MAP, and thus the compiler cannot build a canonical value for constant maps to use in comparisons. (And I don't think there should be a single spec, different implementations have different requirements). So, I think it's acceptable if we don't simplify MAP constants and temporarily disable the simplification capabilities for MAP? I've already removed the simplification for MAP constants. Or do we need to implement the simplification of these three complex types together? -- 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]
