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.
---
