cloud-fan commented on pull request #33751: URL: https://github.com/apache/spark/pull/33751#issuecomment-899580421
> Why are the changes needed? > Make it consistent with the other data type cast. I don't think this is the right reason. Since we turn on ansi interval types by default, I think it's better to avoid breaking changes when not necessary. If the new types are opt-in (like timestamp ntz), then that's a different story. -- 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]
