Mailing-List: contact dev-help@accumulo.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@accumulo.apache.org
Received-SPF: pass (athena.apache.org: local policy includes SPF record at
 spf.trusted-forwarder.org)
MIME-Version: 1.0
In-Reply-To: 
 <CAOyNEs4kHhV_P0yUHCLEb7JyAeCwcrkqeDmtL_v_6E58=9j41g@mail.gmail.com>
References: 
 <CAL5zq9a-Vi9d8Z8YXBR0Hy-JiyiQhFTE9eEagV8qUBNXN-AOcw@mail.gmail.com>
 <CABwccFGRfwtrnByug3Qn52iSTZJXqtBSx51KS1M4qZaD6UUnLg@mail.gmail.com>
 <CAL5zq9axWU++o5LS72XRaSO5T3HbkHM8gemdCR2HGkz3AwNJSQ@mail.gmail.com>
 <CABwccFF9wN9je1GS-=m1pxv2OWgx=581iKVsC-+AecsN-Ha_=g@mail.gmail.com>
 <CAL5zq9a46UoOV43_EdJHUmeY-8LKgxDjbnDOWYSvGT80iYiW_Q@mail.gmail.com>
 <CAJRvFdoTDste36BM5S7cJQs=zP8noev3SLMtpS+njKaS07Nkkg@mail.gmail.com>
 <CAL5zq9bLcbcSGF7T+2dTS6igHyokWjd1yMbNV+gxWfQDK53gng@mail.gmail.com>
 <CAD-fFUJ2JCou_NW9rZrj6S5+8tod4c93SPhnj74PUPqSWgKM2A@mail.gmail.com>
 <CAL5zq9ZR5JtQWy0FuRwf2kjBkwpuD5vC6_RwYbusMw6+jO=BYg@mail.gmail.com>
 <CAGUtCHrnC-4q1rcAy8MMS+njWY=byN9M2dQe0U2kpAqKeS5S9g@mail.gmail.com>
 <CAFAMeY+ceRqPW1vxEXjapg6dJB25kca94VE06RD8AkQDE6xm2w@mail.gmail.com>
 <CAGUtCHrThjFW-u+JK+ZYALB-qut2VNd5AH1Sb8o7UcQFUXM4xA@mail.gmail.com>
 <CAFAMeYK_2b+XJuHJt2s=L9gfkjvWFFD3vP=miNQ+yRnuutWSCQ@mail.gmail.com>
 <CABwccFGTMMHi6gaEcKi-7W-_OuKXFF+xY-OCSz4FEhWB6HgDvA@mail.gmail.com>
 <CAMz+DuvUZrg-2FyoL68+PbLqMn0Y4TKrPf2duvCkVCH-2XxRqg@mail.gmail.com>
 <CAL5zq9ac7mmE6Q-AEiZ30z_abMftH3Hjaq5vToKStFyAX8njHw@mail.gmail.com>
 <CAMz+DuuDQks1zgR6_bJOUzSxUGM70pshbtSxaDYOKVWEZ6xfWw@mail.gmail.com>
 <CAOyNEs4kHhV_P0yUHCLEb7JyAeCwcrkqeDmtL_v_6E58=9j41g@mail.gmail.com>
From: Sean Busbey <busbey+ml@clouderagovt.com>
Date: Mon, 30 Dec 2013 09:05:51 -0600
Message-ID: 
 <CABwccFFWXm0GXN-joOTim044r7aTZSHw-gimJhXqwwJqa2OeJg@mail.gmail.com>
Subject: Re: Resource leak warnings
To: "dev@accumulo apache. org" <dev@accumulo.apache.org>
Content-Type: multipart/alternative; boundary=047d7b603a78d52ec404eec1c84a

--047d7b603a78d52ec404eec1c84a
Content-Type: text/plain; charset=UTF-8

On Mon, Dec 30, 2013 at 8:57 AM, Josh Elser <josh.elser@gmail.com> wrote:

> On Dec 30, 2013 9:36 AM, "William Slacum" <wilhelm.von.cloud@accumulo.net>
> wrote:
> >
> > That being said, has anyone started on the utility so we can at least
> have
> > a comparison/bake off?
>
> A utility that forcefully closes all resources to prevent a leak? I don't
> believe I've seen a ticket for that yet. Please make one if so.
>
> What do you want to compare? That the resources are cleaned up? I would
> assume that would be a part of testing before the aforementioned utility
> would be committed.
>
>

There's no ticket yet. Jared Winick, the creator of the utility to test the
close() based solution, mentioned in chat that he could do it this week.
I'll circle back with him and get a ticket made so that either he or I can
get it done.


> > I assume this is going to block  1.6.0/1.5.1.
> >
>
> Only if we decide it should :). It's one of those things that has likely
> bit people for quite some time. It's up to us to decide if it's severe
> enough that we should try to get it fixed before making another release.
>


Actually, this should block 1.6.0, 1.5.1, and 1.4.5 since once published
we'll be stuck with the API change.

Since this impacts anyone working in a reusable container environment, and
we've already had one group of users bring it up as a significant issue, I
would vote that it should block. Non-binding, that is.

-- 
Sean

--047d7b603a78d52ec404eec1c84a--