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`?---
