Github user twalthr commented on the issue:
https://github.com/apache/flink/pull/5040
Thanks for the update @walterddr. I will merge this...
---
Github user walterddr commented on the issue:
https://github.com/apache/flink/pull/5040
Thanks @twalthr for confirming, I will also add in the documentation today
then.
---
Github user twalthr commented on the issue:
https://github.com/apache/flink/pull/5040
Thanks. I will have a look at it soon. I think it ok that the Table API is
a super set of features in this case.
---
Github user walterddr commented on the issue:
https://github.com/apache/flink/pull/5040
Yes. I think it would also be good to support `row(field, 'literal')` on
both SQL and Table API so I added that.
Not sure if this is necessary but `row(1, 'foo').get('f0')` is also
Github user twalthr commented on the issue:
https://github.com/apache/flink/pull/5040
@walterddr Yes of course. Having this for the Table API as well would be
great. Something like `row(1, "My String")`?
---
Github user walterddr commented on the issue:
https://github.com/apache/flink/pull/5040
Thanks for the prompt review @twalthr . Yeah I think generalizing the
codegen part would be great.
Also do you think I can put in TableAPI support in this PR as well? I will
update the