Mailing-List: contact dev-help@curator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@curator.apache.org
MIME-Version: 1.0
In-Reply-To: <CALuNCpiUqW6xjN8MLze-t2xeQ-ZzW=-tLm3s2-U4iUip5Lt46A@mail.gmail.com>
References: <CANykduZ_8vgz5wp19_am_G7s-CFk7_gN-D0z2Dt1NdxQ5Pjs+g@mail.gmail.com>
	<F65C2749-6585-4188-A2AF-C448AADE7B59@jordanzimmerman.com>
	<CANykduYbUtRzrVEGLzBeMmvkNuSvSRrqgEZy5TOaKT9gQEat_Q@mail.gmail.com>
	<CANykduYtnkom1CePvuMA30K0FPaAJmcC65R4ep7=-nc=vy0Dzg@mail.gmail.com>
	<CANykdubO+0xGNyjTbYWaRgR6ZRb3E7Y11VNMXBZ98NbZXQRCng@mail.gmail.com>
	<9BFB70F4-3965-4711-8838-B9CBC33F1BD0@jordanzimmerman.com>
	<CANykdua5oyFT2duPoS=MFR_EnSuxvtp0kL3OQHFJV7ufag03ag@mail.gmail.com>
	<18A7FA8A-6D57-4861-9740-7DFE5DBF8065@jordanzimmerman.com>
	<CALuNCpgp7Zj1j7FxJsyGCs8HRaNRnt3yci+Gs8UR-69WfnZQgA@mail.gmail.com>
	<CANykduZZ5XJU_d8xD4XQEvouz5jLC-GBAxyLO395tbvyhWmi6Q@mail.gmail.com>
	<CALuNCpiUqW6xjN8MLze-t2xeQ-ZzW=-tLm3s2-U4iUip5Lt46A@mail.gmail.com>
Date: Thu, 26 May 2016 11:47:29 +1000
Message-ID: <CANykduZDJgW3+sPsr_rY84u=-uD53JkQf-Q=WnhiNDZG83Zvxw@mail.gmail.com>
Subject: Re: CURATOR-3.0 tests
From: Cameron McKenzie <mckenzie.cam@gmail.com>
To: "dev@curator.apache.org" <dev@curator.apache.org>
Content-Type: multipart/alternative; boundary=94eb2c056f4e1ade540533b4f9b5
archived-at: Thu, 26 May 2016 01:47:36 -0000

--94eb2c056f4e1ade540533b4f9b5
Content-Type: text/plain; charset=UTF-8

Thanks Scott.

Push the fix to master and merge it into 3.0.

Then I guess, I'll push new versions of 2.11 and 3.2 onto Nexus.
cheers

On Thu, May 26, 2016 at 11:44 AM, Scott Blum <dragonsinth@gmail.com> wrote:

> Alright, I have a fix, but it wants to be applied to both master and 3.0.
> Where should I push the fix?
>
> On Wed, May 25, 2016 at 6:10 PM, Cameron McKenzie <mckenzie.cam@gmail.com>
> wrote:
>
> > Thanks Scott,
> > If you just checkout the CURATOR-3.0 branch, they are failing there.
> > cheers
> >
> > On Thu, May 26, 2016 at 2:06 AM, Scott Blum <dragonsinth@gmail.com>
> wrote:
> >
> > > Sure, what SHA are they failing at Cam?
> > >
> > > On Wed, May 25, 2016 at 9:36 AM, Jordan Zimmerman <
> > > jordan@jordanzimmerman.com> wrote:
> > >
> > > > Scott can you take a look?
> > > >
> > > > -Jordan
> > > >
> > > > > On May 25, 2016, at 4:35 AM, Cameron McKenzie <
> > mckenzie.cam@gmail.com>
> > > > wrote:
> > > > >
> > > > > Tree cache tests are still failing. I've tried a few times but no
> > love:
> > > > >
> > > > > TestTreeCacheEventOrdering>TestEventOrdering.testEventOrdering:151
> > > > actual 6
> > > > > expected -31:
> > > > >
> > > > > I will have a look into what's going on in the morning. Given that
> > > these
> > > > > may take some messing about to fix up, do we just want to vote on
> > > 2.11.0
> > > > > separately, as that is all ready to go?
> > > > > cheers
> > > > >
> > > > > On Wed, May 25, 2016 at 5:34 PM, Jordan Zimmerman <
> > > > > jordan@jordanzimmerman.com> wrote:
> > > > >
> > > > >> Great news. Thanks.
> > > > >>
> > > > >> ====================
> > > > >> Jordan Zimmerman
> > > > >>
> > > > >>> On May 25, 2016, at 12:37 AM, Cameron McKenzie <
> > > mckenzie.cam@gmail.com
> > > > >
> > > > >> wrote:
> > > > >>>
> > > > >>> I have fixed up the test case, all good now.
> > > > >>>
> > > > >>> On Wed, May 25, 2016 at 1:45 PM, Cameron McKenzie <
> > > > >> mckenzie.cam@gmail.com>
> > > > >>> wrote:
> > > > >>>
> > > > >>>> Looks like it was introduced with the schema validation stuff.
> It
> > > now
> > > > >> does
> > > > >>>> an ACL check prior to the backgrounding call. Because the unit
> > test
> > > > >> uses a
> > > > >>>> bogus ACL provider it just throws an exception
> > > > >>>>
> > > > >>>>       final String adjustedPath =
> > > > >>>> adjustPath(client.fixForNamespace(givenPath,
> > > > >> createMode.isSequential()));
> > > > >>>>       List<ACL> aclList = acling.getAclList(adjustedPath);
> > > > >>>>
> > > > >>>>
> > > client.getSchemaSet().getSchema(givenPath).validateCreate(createMode,
> > > > >> data,
> > > > >>>> aclList);
> > > > >>>>
> > > > >>>>       String returnPath = null;
> > > > >>>>       if ( backgrounding.inBackground() )
> > > > >>>>       {
> > > > >>>>           pathInBackground(adjustedPath, data, givenPath);
> > > > >>>>
> > > > >>>> So, I guess the answer is to get the test to force a failure in
> a
> > > > >>>> different way. With the UnhandledErrorListener, the expectation
> is
> > > > that
> > > > >>>> this only gets called on backgrounding operations?
> > > > >>>> cheers
> > > > >>>>
> > > > >>>> On Wed, May 25, 2016 at 1:39 PM, Cameron McKenzie <
> > > > >> mckenzie.cam@gmail.com>
> > > > >>>> wrote:
> > > > >>>>
> > > > >>>>> Same on the master branch, but it passes there, so maybe
> > something
> > > > has
> > > > >>>>> legitimately broken the test. Will let you know if I get stuck.
> > > > >>>>> cheers
> > > > >>>>>
> > > > >>>>> On Wed, May 25, 2016 at 1:36 PM, Jordan Zimmerman <
> > > > >>>>> jordan@jordanzimmerman.com> wrote:
> > > > >>>>>
> > > > >>>>>> Let me know if you need help.
> > > > >>>>>>
> > > > >>>>>> It might be a bad merge. Have you compared it to the master
> > > branch?
> > > > >>>>>>
> > > > >>>>>> -JZ
> > > > >>>>>>
> > > > >>>>>>>> On May 24, 2016, at 10:31 PM, Cameron McKenzie <
> > > > >> mckenzie.cam@gmail.com>
> > > > >>>>>>> wrote:
> > > > >>>>>>>
> > > > >>>>>>> Guys,
> > > > >>>>>>> There's a test TestFrameworkBackground:testErrorListener that
> > is
> > > > >>>>>> failing
> > > > >>>>>>> consistently on the CURATOR-3.0 branch.
> > > > >>>>>>>
> > > > >>>>>>> I can't see how it has ever worked. It seems to try and
> provoke
> > > an
> > > > >>>>>> error
> > > > >>>>>>> via a bad ACL provider.
> > > > >>>>>>>
> > > > >>>>>>> But this ACL provider is called by the CreateBuilderImpl
> prior
> > to
> > > > the
> > > > >>>>>>> backgrounding call, which means that the exception that it
> > throws
> > > > >>>>>> happens
> > > > >>>>>>> in the main Thread of the unit test. So, it just throws an
> > > > >>>>>>> UnsupportedOperationException which is propogated up the
> stack
> > at
> > > > >> which
> > > > >>>>>>> point the unit test fails.
> > > > >>>>>>>
> > > > >>>>>>> So, I will look at fixing this, but I just don't understand
> how
> > > it
> > > > >> ever
> > > > >>>>>>> worked?
> > > > >>>>>>> cheers
> > > > >>>>
> > > > >>
> > > >
> > > >
> > >
> >
>

--94eb2c056f4e1ade540533b4f9b5--