accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Sean Busbey <busbey...@clouderagovt.com>
Subject Re: Resource leak warnings
Date Fri, 13 Dec 2013 21:34:47 GMT
On Fri, Dec 13, 2013 at 3:26 PM, Christopher <ctubbsii@apache.org> wrote:

> On Fri, Dec 13, 2013 at 3:55 PM, Sean Busbey <busbey+ml@clouderagovt.com>
> wrote:
> > I think the answer should be #3 and I am willing to find time to
> implement
> > it next week.
>
> That's great. In the meantime, I'm doing #4, so I can clear the noise
> and focus on my own tickets.
>
>
Could you just do #4 as a patch on your own local repo? It should be
relatively easy to keep rebased given that it's just an interface.



>
> > Also, you should not be able to use a Connection that is attached to a
> > now-closed Instance. I would also consider such use a bug.
> >
> [snip]
>
> It's not just the Connection you'd have to enforce not being re-used
> after the Instance is closed... it's every object our factory-based
> API can give you, all the way down. That's not a small change.
>
>
I agree that fixing things to always expose the bug would be hard. I don't
think that changes that it's a bug.


-- 
Sean

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message