dianfu commented on a change in pull request #14758:
URL: https://github.com/apache/flink/pull/14758#discussion_r566698903



##########
File path: flink-python/pyflink/fn_execution/operation_utils.py
##########
@@ -310,7 +310,7 @@ def wrapped_keyed_process_function(value):
             on_timer_ctx.set_timestamp(value[1])
             on_timer_ctx.timer_service().set_current_watermark(value[2])
             current_key = value[3]
-            on_timer_ctx.set_current_key(current_key)
+            on_timer_ctx.set_current_key(current_key[0])

Review comment:
       Is this a change specific for this PR or is it an issue which also 
exists in 1.12? If this is the latter case, we'd better split it into a 
separate commit or separate PR and also cherry pick to 1.12. What do you think?




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to