matriv commented on pull request #17439: URL: https://github.com/apache/flink/pull/17439#issuecomment-940996977
@twalthr I changed the `fail` to `failTableApi` to make it visible that it's only for that. - Should I keep a comment pointing to the same issue since we want to test also for SQL once calcite PR is merged and adopted, or create a new issue just for that (and still keep a comment pointing to it)? - Please don't forget that this is a breaking change since currently those casts where allowed (proper label?) -- 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]
