danny0405 edited a comment on pull request #13050:
URL: https://github.com/apache/flink/pull/13050#issuecomment-668928645


   > Could you name a few of these breaking changes? I know about the dropped 
windows but what else?
   
   I listed some features i though important (very probably breaking) here:
   
   - [CALCITE-3786] The rel node digest was refactored
   - [CALCITE-3896 Top down trait pass through
   - [CALCITE-3916 Support cascades style top-down driven rule apply
   - [CALCITE-2569] UDFs that are table functions must implement 
SqlTableFunction and have CURSOR as their return type
   - [CALCITE-3923] Refactor how planner rules are parameterized
   
   But anyway, Calcite is a single open source community and we can not always 
depend on the version upgrade, and Calcite does not always evolve following 
Flink's request. If we have a simple workaround for a quick fix(log an issue 
there and remove the code when upgrade), it is better than just wait for the 
upgrade.
   
   This fix is critical, user can not even use the table function now in a 
`CREATE VIEW` and is a blocker for VVP platform.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to