Github user fhueske commented on the issue:
https://github.com/apache/flink/pull/3269
Hi @tonycox, thanks for the update!
I'll do some minor improvements and will merge the PR.
Thank you,
Fabian
---
If your project is set up for it, you can reply to this email
Github user tonycox commented on the issue:
https://github.com/apache/flink/pull/3269
@fhueske What dou you think about this PR?
---
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
Github user tonycox commented on the issue:
https://github.com/apache/flink/pull/3269
Hi @fhueske I've addressed all comments
---
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
Github user tonycox commented on the issue:
https://github.com/apache/flink/pull/3269
Hi @wuchong
Thank you for excellent advice
---
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
Github user wuchong commented on the issue:
https://github.com/apache/flink/pull/3269
Hi @tonycox , thanks for this PR.
It seems that you are using column names to represent the (nested) fields
that are projected. I'm afraid that it can't work when the composite columns