dawidwys commented on code in PR #23735:
URL: https://github.com/apache/flink/pull/23735#discussion_r1395745935
##########
flink-table/flink-table-common/src/main/java/org/apache/flink/table/types/inference/strategies/SpecificInputTypeStrategies.java:
##########
@@ -65,6 +65,11 @@ public static InputTypeStrategy
windowTimeIndicator(TimestampKind timestampKind)
return new WindowTimeIndictorInputTypeStrategy(timestampKind);
}
+ /** See {@link WindowTimeIndictorInputTypeStrategy}. */
+ public static InputTypeStrategy windowTimeIndicator() {
+ return new WindowTimeIndictorInputTypeStrategy(null);
Review Comment:
When we pass it as null? The parameter is used to configure the behaviour of
the `WindowTimeIndictorInputTypeStrategy` class. If we configure it with `null`
we don't perform the specific checks for `PROCTIME`, `ROWTIME` attributes. We
just check that the argument is of any of the two.
I use this in `WINDOW_START` and `WINDOW_END`, I only added comments to make
this cleaner.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
