Github user wuchong commented on the issue:
https://github.com/apache/flink/pull/4200
Thanks. I will merge this.
---
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
Github user Xpray commented on the issue:
https://github.com/apache/flink/pull/4200
I update the code, once calcite-1860 fixed, the ignored test case will pass
---
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
Github user wuchong commented on the issue:
https://github.com/apache/flink/pull/4200
Thanks for updating and rebasing, the changes look good to me now.
+1 to merge
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
