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

["Stefan Rijnhart (JIRA)" <ji...@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 commented on PROTON-1394:
-----------------------------------------

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.


> 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: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org