Roman Khachatryan created FLINK-20107:
-----------------------------------------
Summary: Issues with setting InputChannel.statePersister
Key: FLINK-20107
URL: https://issues.apache.org/jira/browse/FLINK-20107
Project: Flink
Issue Type: Bug
Components: Runtime / Checkpointing, Runtime / Network
Affects Versions: 1.12.0
Reporter: Roman Khachatryan
Assignee: Roman Khachatryan
Fix For: 1.12.0
ChannelStatePersister is a non-final field in InputChannel.
The need for injection is caused by different places, where network- and state-
related objects are created (task executor vs stream task).
It is set on remote and local channels upon requesting partitions, after
converting unknown or recovered channel.
Issues
# Not set on RemoteInputChannel when converting from UnknownChannel
# No visibility guarantee: written by task thread without any (explicit)
synchronization, read by network thread (checkForBarrier)
I see that "final" channels (both remote and local) are created only when the
writer is known (partitions requested).
So we can just make it final in "final" channels and pass from
recovered/unknown. For that,
* need to add to UnknownChannel
* no need to make it volatile/guarded because in "non-final" channels it's
accessed only by the task thread; and in "final" channels it will be final
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
