[jira] [Commented] (BEAM-3087) Extend lock scope in Flink UnboundedSourceWrapper

["Kenneth Knowles (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/BEAM-3087?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16339655#comment-16339655 ] 

Kenneth Knowles commented on BEAM-3087:
---------------------------------------

Does this need to block 2.3.0? Instead of using the Fix Version as a priority list, I have bumped up the priority. What are the consequences of this bug?

> Extend lock scope in Flink UnboundedSourceWrapper
> -------------------------------------------------
>
>                 Key: BEAM-3087
>                 URL: https://issues.apache.org/jira/browse/BEAM-3087
>             Project: Beam
>          Issue Type: Bug
>          Components: runner-flink
>            Reporter: Aljoscha Krettek
>            Priority: Critical
>             Fix For: 2.3.0
>
>
> In {{UnboundedSourceWrapper}} the lock scope is not big enough: we synchronise in {{emitElement()}} but should instead synchronise inside the reader loop in {{run()}} because the {{Source}} interface does not allow concurrent calls.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)