[ https://issues.apache.org/jira/browse/FLINK-8560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16363704#comment-16363704 ]
ASF GitHub Bot commented on FLINK-8560: --------------------------------------- Github user pnowojski commented on a diff in the pull request: https://github.com/apache/flink/pull/5481#discussion_r168122242 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/GroupAggProcessFunction.scala --- @@ -162,7 +162,7 @@ class GroupAggProcessFunction( override def onTimer( timestamp: Long, - ctx: ProcessFunction[CRow, CRow]#OnTimerContext, + ctx: ProcessFunction[CRow, CRow]#OnTimerContext[_], --- End diff -- I'm not a scala expert, but is this change somehow related to adding the `getCurrentKey()` method? > Access to the current key in ProcessFunction#OnTimerContext after keyBy() > ------------------------------------------------------------------------- > > Key: FLINK-8560 > URL: https://issues.apache.org/jira/browse/FLINK-8560 > Project: Flink > Issue Type: Improvement > Components: DataStream API > Affects Versions: 1.4.0 > Reporter: Jürgen Thomann > Assignee: Bowen Li > Priority: Major > Fix For: 1.5.0 > > > Currently it is required to store the key of a keyBy() in the processElement > method to have access to it in the OnTimerContext. > This is not so good as you have to check in the processElement method for > every element if the key is already stored and set it if it's not already set. > A possible solution would adding OnTimerContext#getCurrentKey() or a similar > method. Maybe having it in the open() method could maybe work as well. > http://apache-flink-user-mailing-list-archive.2336050.n4.nabble.com/Getting-Key-from-keyBy-in-ProcessFunction-tt18126.html -- This message was sent by Atlassian JIRA (v7.6.3#76005)