Re: [Twisted-Python] deferToThread and thrown exceptions break gc - ticket 3853

2013-06-26 Thread exarkun
On 08:15 pm, rut...@osu.edu wrote: I am not sure that that trac issue contains a patch that will fix it. The most recent patch on that trac issue is for twisted 8.1, and it does not apply cleanly to twisted 12.3.0. When I tried to place the 2 lines that form the patch in the most obvious place

Re: [Twisted-Python] deferToThread and thrown exceptions break gc - ticket 3853

2013-06-26 Thread Benjamin Rutt
I am not sure that that trac issue contains a patch that will fix it. The most recent patch on that trac issue is for twisted 8.1, and it does not apply cleanly to twisted 12.3.0. When I tried to place the 2 lines that form the patch in the most obvious place in the 12.3.0 codebase (before the co

Re: [Twisted-Python] deferToThread and thrown exceptions break gc - ticket 3853

2013-06-26 Thread Christopher Armstrong
On Wed, Jun 26, 2013 at 9:15 AM, Benjamin Rutt wrote: > Hi, > > I believe I hit http://twistedmatrix.com/trac/ticket/3853 in production > this week, in the form of what looked like a resource leak, but turned out > to be a case of a resource held for much longer than expected. (I'm using > pytho

[Twisted-Python] deferToThread and thrown exceptions break gc - ticket 3853

2013-06-26 Thread Benjamin Rutt
Hi, I believe I hit http://twistedmatrix.com/trac/ticket/3853 in production this week, in the form of what looked like a resource leak, but turned out to be a case of a resource held for much longer than expected. (I'm using python 2.7, and twisted 12.3.0). That is, I had a function invoked by d