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

Posted to issues@flink.apache.org by haohui <gi...@git.apache.org> on 2017/09/24 07:42:47 UTC

[GitHub] flink issue #4710: [FLINK-7446] [table] Change DefinedRowtimeAttribute to wo...

Github user haohui commented on the issue:

    https://github.com/apache/flink/pull/4710
  
    LGTM overall +1.
    
    One question: since we now cast `ROWTIME` / `PROCTIME` directly to `LONG`, I wonder, do we want to revisit the decision that creates dedicated types for `ROWTIME` / `PROCTIME`?


---