You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2020/05/04 09:48:50 UTC

[GitHub] [flink] dawidwys commented on pull request #11951: [FLINK-17420][table sql / api]Cannot alias Tuple and Row fields when converting DataStream to Table

dawidwys commented on pull request #11951:
URL: https://github.com/apache/flink/pull/11951#issuecomment-623366784


   Can not tell for sure as this code is quite old (It was ported in 1.9 from `TableEnvironment.scala`). It worked as you are describing that you could/can not use the `rowtime` in by position reference mode.
   
   In my opinion it does make sense it is not possible to use it in the by position mode. Because what does the `a` in `$("a").rowtime().as("rowtime")` actually means in that case? I also find the syntax with `proctime` a bit confusing (I mentioned in the ML thread that I don't like a few aspects of the method). In proctime the original field is also ignored which is weird. Nevertheless I would not change it now.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org