[ https://issues.apache.org/jira/browse/FLINK-4953?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15962178#comment-15962178 ]
ASF GitHub Bot commented on FLINK-4953: --------------------------------------- Github user aljoscha commented on the issue: https://github.com/apache/flink/pull/3661 I'm currently in SF for http://sf.flink-forward.org and quite busy. That's why my responses are quite slow, sorry for that. A quick answer: There is the (slightly hard to find) `InternalWindowFunctionTest` which tests that the internal window functions correctly forward calls to a process window function and other (user facing) window functions. We should test the internal logic of the new time access code for `InternalWindowFunction` in `WindowOperatorContractTest`, (see, for example `testPerWindowStateSetAndClearedOnEventTimePurge()`). We should add more tests that verify that per-window state and also the new time access is correctly forwarded in `InternalWindowFunctionTest`. What do you think? > Allow access to "time" in ProcessWindowFunction.Context > ------------------------------------------------------- > > Key: FLINK-4953 > URL: https://issues.apache.org/jira/browse/FLINK-4953 > Project: Flink > Issue Type: Improvement > Components: DataStream API > Reporter: Manu Zhang > Assignee: Manu Zhang > > The recently added {{ProcessWindowFunction}} has a {{Context}} object that > allows querying some additional information about the window firing that we > are processing. Right now, this is only the window for which the firing is > happening. We should extends this with methods that allow querying the > current processing time and the current watermark. > Original text by issue creator: This is similar to FLINK-3674 but exposing > time information in window functions. Currently when a timer is fired, all > states in a window will be returned to users, including those after the > timer. This change will allow users to filter out states after the timer > based on time info. -- This message was sent by Atlassian JIRA (v6.3.15#6346)