Mailing-List: contact dev-help@curator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@curator.apache.org
Date: Tue, 19 May 2015 22:37:52 -0700
From: Jordan Zimmerman <jordan@jordanzimmerman.com>
To: Cameron McKenzie <mckenzie.cam@gmail.com>
Cc: "=?utf-8?Q?dev=40curator.apache.org?=" <dev@curator.apache.org>
Message-ID: <etPan.555c1db0.6bb07a80.1fdb@Jordans-MacBook-Pro-2.local>
In-Reply-To: 
 <CANykduZPABvEcw2wtJGCgT46hkyqvSLMn580686S72N11T4N2A@mail.gmail.com>
References: 
 <CANykdub2rBpGNY5C0uVg39-77jPm3fupuRn0HO6sMoZkPkkZXw@mail.gmail.com>
 <etPan.555bc470.507ed7ab.3e9@jordans-mbp-2.bluejeansnet.com>
 <CANykduaXEk-ZOw0v0qxKYFXWV1V6Jg0MFE_7tQEhe0H96EKr9A@mail.gmail.com>
 <etPan.555bc5de.7545e146.3e9@jordans-mbp-2.bluejeansnet.com>
 <CANykduZPABvEcw2wtJGCgT46hkyqvSLMn580686S72N11T4N2A@mail.gmail.com>
Subject: Re: CURATOR-161
MIME-Version: 1.0
Content-Type: multipart/alternative; boundary="555c1db0_5a1e8b91_1fdb"

--555c1db0_5a1e8b91_1fdb
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline

Sure - that=E2=80=99s fine. I=E2=80=99ll do that


On May 19, 2015 at 9:22:32 PM, Cameron McKenzie (mckenzie.cam=40gmail.com=
) wrote:

Jordan,
I noticed that in CURATOR-217 you fixed up the license headers of a bunch=
 of the files that were committed as part of CURATOR-161. I don't want to=
 merge CURATOR-161 back to the 3.0 branch without the headers as I think =
it will break the build. So, what's the best way forward=3F Do you just w=
ant to merge CURATOR-217 back to the 3.0 branch given that it already con=
tains all of the CURATOR-161 changes=3F
cheers

On Wed, May 20, 2015 at 9:23 AM, Jordan Zimmerman <jordan=40jordanzimmerm=
an.com> wrote:
https://github.com/apache/curator/pull/82


On May 19, 2015 at 4:18:36 PM, Cameron McKenzie (mckenzie.cam=40gmail.com=
) wrote:

Thanks Jordan,
Will have a look at the PR.
cheers

On Wed, May 20, 2015 at 9:17 AM, Jordan Zimmerman <jordan=40jordanzimmerm=
an.com> wrote:
I=E2=80=99ll have a look. Also, I already started on integrating watcher =
removal as I have strong ideas on it. Here=E2=80=99s the current PR:=C2=A0=
https://github.com/apache/curator/pull/80

-JZ


On May 19, 2015 at 4:06:03 PM, Cameron McKenzie (cammckenzie=40apache.org=
) wrote:

Guys,
If anyone's got a spare few minutes could they please review the final
changes to CURATOR-161. If we're happy with the implementation then I'll
merge into the 3.0 branch and then start work on getting the recipes to u=
se
the watcher removal stuff.
cheers
Cam


--555c1db0_5a1e8b91_1fdb--