flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From fhueske <...@git.apache.org>
Subject [GitHub] flink issue #4710: [FLINK-7446] [table] Change DefinedRowtimeAttribute to wo...
Date Tue, 26 Sep 2017 17:54:15 GMT
Github user fhueske commented on the issue:

    https://github.com/apache/flink/pull/4710
  
    Thanks for the review @twalthr.
    I've updated the PR. 
    
    @haohui: This PR preserves the current logic that time attributes are exposed as `TIMESTAMP`.
I agree that support for time indicators that expose themselves as `Long` is desirable. However,
this requires quite a few changes as we need to extend several functions (incl. `TUMBLE`,
`HOP`, etc.) and validation logic in some operators (over windows, joins, etc.). So this is
not a lightweight change and should be done as a separate issue, IMO. 


---

Mime
View raw message