You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geode.apache.org by nabarunnag <gi...@git.apache.org> on 2017/03/19 17:37:10 UTC

[GitHub] geode pull request #430: GEODE-2690: Created a new thread pool for flush ope...

GitHub user nabarunnag opened a pull request:

    https://github.com/apache/geode/pull/430

    GEODE-2690: Created a new thread pool for flush operations

    	* WaitUntilParallelGatewaySenderFlushedCoordinator's waitUntilFlushed will use a separate thread pool.
    	* This new thread pool is limited to have only 100 threads
    
    @boglesby @upthewaterspout @jhuynh1 @gesterzhou 
    
    This is one of the solution which makes flush operations to use a thread pool limited to 100 threads.
    Using this fix we are able to see that during tests, we do not encounter issues like unable to create native threads.

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

    $ git pull https://github.com/nabarunnag/incubator-geode feature/GEODE-2690

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

    https://github.com/apache/geode/pull/430.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 #430
    
----
commit 5e88d52d6dce57a36f014c0073e53bc7f69902f5
Author: nabarun <nn...@pivotal.io>
Date:   2017-03-19T04:15:05Z

    GEODE-2690: Created a new thread pool for flush operations
    
    	* WaitUntilParallelGatewaySenderFlushedCoordinator's waitUntilFlushed will use a separate thread pool.
    	* This new thread pool is limited to have only 100 threads

----


---
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.
---

[GitHub] geode pull request #430: GEODE-2690: Created a new thread pool for flush ope...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/geode/pull/430


---
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.
---

[GitHub] geode issue #430: GEODE-2690: Created a new thread pool for flush operations

Posted by upthewaterspout <gi...@git.apache.org>.
Github user upthewaterspout commented on the issue:

    https://github.com/apache/geode/pull/430
  
    I Think it would be better to figure out why we a creating a ton of threads, rather than create a thread pool specifically just for this one message.


---
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.
---