GitHub user patricklucas opened a pull request:
https://github.com/apache/incubator-storm/pull/49
Add locking in acquire-random-range-id
This fixes a threadsafety issue documented in STORM-120. This is an
acceptable temporary fix, but the cause of concurrent access to this
variable should still be identified.
@revans2 suggested this fix.
Refs STORM-120
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/patricklucas/incubator-storm
STORM-120_fix_executor_threadsafety
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-storm/pull/49.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #49
----
commit ac993b43db57e8971a9a662c208f8ac07571e44d
Author: Patrick Lucas <[email protected]>
Date: 2014-03-21T20:33:56Z
Add locking in acquire-random-range-id
This fixes a threadsafety issue documented in STORM-120. This is an
acceptable temporary fix, but the cause of concurrent access to this
variable should still be identified.
Refs STORM-120
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
