[GitHub] flink issue #4373: [FLINK-6429] [table] Bump up Calcite version to 1.13.

[dianfu <gi...@git.apache.org>]

Github user dianfu commented on the issue:

    https://github.com/apache/flink/pull/4373
  
    I have did some investigation of the test failure of **JoinITCase.testJoinWithExpressionPreds** and would like to share my findings and solutions for your reference. The cause of this issue is that for preserved expressions, in **PushProjector#createProjectRefsAndExprs**, the column names corresponding to them will be the operator names of the expressions. For example for expression **a - 1** in the test case, the  column corresponding to it will be **-**. I think this behavior is not expected and have copied **PushProjector** from calcite and made some changes to it (line 507). Please refer to [here](https://github.com/dianfu/flink/commit/efa9641e0bd395a3679b0d496b60e3d42aa7b832) for more information.
    
    For the files copied from calcite, all files can be removed except **SqlTimestampAddFunction** and **AuxiliaryConverter**.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---