[jira] [Comment Edited] (PROTON-1394) Creating a Container leaks two file descriptors

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[jira] [Comment Edited] (PROTON-1394) Creating a Container leaks two file descriptors

JIRA jira@apache.org

    [ https://issues.apache.org/jira/browse/PROTON-1394?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16051770#comment-16051770 ]

Stefan Rijnhart edited comment on PROTON-1394 at 6/16/17 11:10 AM:
-------------------------------------------------------------------

Thanks for the patch. Seems these two lines in _execute_on_error() are redundant

{{    else:
        on_error = on_error
}}


We are currently having a look at https://github.com/twang817/weakrefmethod to include this fix using Python 2.



was (Author: stefanrijnhart):
Thanks for the patch. Seems these two lines in _execute_on_error() are redundant

{{
{quote}      else:
        on_error = on_error{quote}}}


We are currently having a look at https://github.com/twang817/weakrefmethod to include this fix using Python 2.


> Creating a Container leaks two file descriptors
> -----------------------------------------------
>
>                 Key: PROTON-1394
>                 URL: https://issues.apache.org/jira/browse/PROTON-1394
>             Project: Qpid Proton
>          Issue Type: Bug
>          Components: python-binding
>    Affects Versions: 0.16.0
>            Reporter: Mike Bonnet
>         Attachments: fix_container_leak.patch, leakyprotonpipes.py, new_fix_container_leak.patch
>
>
> Creating a Container (Reactor) creates a pipe (two file descriptors). This pipe is never freed, even after the Container is stopped and goes out of scope. An application that creates many short-lived Containers will quickly exhaust file descriptors and Container creation will start failing.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Loading...