You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@storm.apache.org by "Jungtaek Lim (JIRA)" <ji...@apache.org> on 2018/04/10 09:10:00 UTC

[jira] [Resolved] (STORM-3020) Fix race condition is async localizer

     [ https://issues.apache.org/jira/browse/STORM-3020?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jungtaek Lim resolved STORM-3020.
---------------------------------
       Resolution: Fixed
    Fix Version/s: 2.0.0

Thanks [~revans2], I merged into master.

> Fix race condition is async localizer
> -------------------------------------
>
>                 Key: STORM-3020
>                 URL: https://issues.apache.org/jira/browse/STORM-3020
>             Project: Apache Storm
>          Issue Type: Bug
>          Components: storm-server
>    Affects Versions: 2.0.0
>            Reporter: Robert Joseph Evans
>            Assignee: Robert Joseph Evans
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 2.0.0
>
>          Time Spent: 1h 10m
>  Remaining Estimate: 0h
>
> I think this impacts all of the code that uses asynclocalizer, but I need to check to be sure.
> As part of a review of a different pull request against AsyncLocalizer I noticed that requestDownloadTopologyBlobs is synchronized, but everything it does is async, but there is a race in one of the async pieces where we read from a map, and then try to update the map later, all outside of a lock.



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