average-vr-56795
10/30/2019, 4:21 PMSession
is a cheap wrapper around an Arc<InnerSession>
which we clone and drop all over the place; InnerSession
is the thing that has the correct lifetimehundreds-breakfast-49010
10/30/2019, 10:38 PMDrop
on InnerSession
, and just have the drop method print a line, I find that that line never gets printedaverage-vr-56795
10/31/2019, 9:04 AMSession
object, or it’s referenced in a cycle somewhere, or something?session_destroy
where we expect to be done with the Session
on the Python side, rather than relying on ffi.gc
to call it?hundreds-breakfast-49010
10/31/2019, 5:16 PMSession
for the latest workunts purely in pythonaverage-vr-56795
10/31/2019, 5:35 PMLocalPantsRunner._run
creates (via one of its method calls) the Session
, and the Session
can be cleaned up before that method returns