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

2017-08-08 Thread twalthr
Github user twalthr commented on the issue: https://github.com/apache/flink/pull/4373 Thanks for the update @haohui. I think this PR is in a good shape now. I will go over the changes again and merge this. Unfortunately, the new findbugs plugin causes the build to fail. We have to wai

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

2017-08-01 Thread haohui
Github user haohui commented on the issue: https://github.com/apache/flink/pull/4373 @dianfu thanks for the investigation! It turns out that it no longer requires copying {{PushProjector}} after rebasing to the latest tip. Updated the PR to address the issue. --- If your project is

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

2017-08-01 Thread dianfu
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 fo