You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@couchdb.apache.org by rnewson <gi...@git.apache.org> on 2015/07/13 20:29:21 UTC

[GitHub] couchdb-couch pull request: Erase async_open entry in failure case

GitHub user rnewson opened a pull request:

    https://github.com/apache/couchdb-couch/pull/72

    Erase async_open entry in failure case

    COUCHDB-2749

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

    $ git pull https://github.com/cloudant/couchdb-couch 2749-couch_server-leak

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

    https://github.com/apache/couchdb-couch/pull/72.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 #72
    
----
commit 5406e57569b71f27af0818a857b41381faba4215
Author: Robert Newson <rn...@apache.org>
Date:   2015-07-13T18:28:08Z

    Erase async_open entry in failure case
    
    COUCHDB-2749

----


---
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] couchdb-couch pull request: Track open time without using process ...

Posted by robertkowalski <gi...@git.apache.org>.
Github user robertkowalski commented on the pull request:

    https://github.com/apache/couchdb-couch/pull/72#issuecomment-121038136
  
    @rnewson can you add some context to the commit message so you don't have to search the issue in jira to understand the issue when reading the git log?
    
    e.g.
    
    ```
    Track open time without using process dictionary
    
    We only remove the `{async_open, DbName}` process dictionary entries if 
    the open is successful. Over time, in busy situations, the process dictionary
    can grow very large.
    
    ```


---
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] couchdb-couch pull request: Track open time without using process ...

Posted by kxepal <gi...@git.apache.org>.
Github user kxepal commented on the pull request:

    https://github.com/apache/couchdb-couch/pull/72#issuecomment-121022690
  
    +1


---
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] couchdb-couch pull request: Track open time without using process ...

Posted by robertkowalski <gi...@git.apache.org>.
Github user robertkowalski commented on the pull request:

    https://github.com/apache/couchdb-couch/pull/72#issuecomment-121038289
  
    it will also help us creating the next release log for 2.0 :)


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