[FLINK-7659] Unprotected access to inProgress in JobCancellationWithSavepointHandlers#handleNewRequest

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.3.4, 1.4.0
• Component/s: Runtime / REST
• Assignee: Bowen Li
• Reporter: Ted Yu
• Created: 20/Sep/17 21:05
• Updated: 12/Mar/18 15:41
• Resolved: 12/Mar/18 15:41

Description

Here is related code:

              } finally {
                inProgress.remove(jobId);
              }

A little lower, in another finally block, there is:

          synchronized (lock) {
            if (!success) {
              inProgress.remove(jobId);

which is correct.

Issue Links

• is duplicated by: FLINK-7433 Lack of synchronization accessing inProgress in JobCancellationWithSavepointHandlers
• links to: GitHub Pull Request #4703