You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2017/07/03 09:51:00 UTC
[jira] [Commented] (FLINK-6310) LocalExecutor#endSession() uses
wrong lock for synchronization
[ https://issues.apache.org/jira/browse/FLINK-6310?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16072189#comment-16072189 ]
ASF GitHub Bot commented on FLINK-6310:
---------------------------------------
Github user aljoscha commented on the issue:
https://github.com/apache/flink/pull/4184
Thanks for fixing this, @zjureel. 👍
I merged, could you please close this PR?
> LocalExecutor#endSession() uses wrong lock for synchronization
> --------------------------------------------------------------
>
> Key: FLINK-6310
> URL: https://issues.apache.org/jira/browse/FLINK-6310
> Project: Flink
> Issue Type: Bug
> Components: Local Runtime
> Reporter: Ted Yu
> Assignee: Fang Yong
> Fix For: 1.4.0
>
>
> Here is related code:
> {code}
> public void endSession(JobID jobID) throws Exception {
> synchronized (LocalExecutor.class) {
> LocalFlinkMiniCluster flink = this.flink;
> {code}
> In other places, lock field is used for synchronization:
> {code}
> public void start() throws Exception {
> synchronized (lock) {
> {code}
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)