cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From miguelaferreira <...@git.apache.org>
Subject [GitHub] cloudstack pull request: Implement a NSX API request execution cou...
Date Mon, 28 Dec 2015 15:55:57 GMT
GitHub user miguelaferreira opened a pull request:

    https://github.com/apache/cloudstack/pull/1294

    Implement a NSX API request execution counter per thread

    The NSX plugin has a execution counter to prevent infinite recursion (and as a result
a stack overflow exception). However, the thread safeness of this counter are not as desired.
The counter was implemented with an AtomicInteger which make it safe for multiple threads
to update and read it. The desired property would be to have a counter per thread.
    
    This PR addresses that issue.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/miguelaferreira/cloudstack nsx-execution-counter-per-thread

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/1294.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 #1294
    
----
commit 40501518b7fbb71fc138a12fdf2d31a56d822619
Author: Miguel Ferreira <miguelferreira@me.com>
Date:   2015-12-17T15:29:59Z

    Add test to check that each thread has it's own execution counter

commit 7f332d760bc824664f41133eeb694fae0f57f5f3
Author: Miguel Ferreira <miguelferreira@me.com>
Date:   2015-12-17T18:47:08Z

    Fix execution counter to support separate counts per thread

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message