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