[GitHub] [flink] shuiqiangchen commented on a change in pull request #14758: [FLINK-21112][python] Add ValueState/ListState/MapState and corresponding StateDescriptors for Python DataStream API

[GitBox <gi...@apache.org>]

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



##########
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:
       The `current_key` is a Row that is wrapped in java operator before sending it to python process, therefore we need to unwrap it before setting to the `on_timer_ctx`. 




----------------------------------------------------------------
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:
users@infra.apache.org