dawidwys commented on pull request #13050: URL: https://github.com/apache/flink/pull/13050#issuecomment-669007993
> 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. I mostly agree. I am a bit sceptical though in that particular case for the following reason: 1. That's not a simple fix, that will be backported back to Calcite. It is a workaround that disables a feature (expanding identifiers) for the `LATERAL` to work. 2. The fix is not in Calcite classes. We must remember to remove this workaround when upgrading Calcite. It's not as simple as dropping Calcite classes. 3. We do not need to wait for Calcite to implement that. If I understand it correctly, the fix is already released. ---------------------------------------------------------------- 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
