Mailing-List: contact dev-help@hbase.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@hbase.apache.org
From: Andrew Purtell <andrew.purtell@gmail.com>
Content-Type: text/plain;
	charset=us-ascii
Content-Transfer-Encoding: quoted-printable
Mime-Version: 1.0 (1.0)
Date: Tue, 30 May 2017 10:13:53 -0700
Subject: Re: VOTE: Merge new Assignment Manager AMv2), HBASE-14614
Message-Id: <E6EA74FF-D795-42A6-91AE-0DD491A846F9@gmail.com>
References: <CADcMMgG7mOHMV52D5BGrWHLY6tp-z=+wT9uvGOPdyQEcskbGQA@mail.gmail.com>
In-Reply-To: <CADcMMgG7mOHMV52D5BGrWHLY6tp-z=+wT9uvGOPdyQEcskbGQA@mail.gmail.com>
To: dev@hbase.apache.org
archived-at: Tue, 30 May 2017 17:14:01 -0000

+1 for merge after all unit tests are passing. No problem to exclude known b=
roken tests not working independent of this patch-set.=20


> On May 30, 2017, at 9:09 AM, Stack <stack@duboce.net> wrote:
>=20
> Moving on from the DISCUSSION thread [4], lets vote on whether to merge th=
e
> new Assignment Manager (AMv2) out on branch HBASE-14614 to master.
>=20
> Unit tests pass. Tests at scale show that the new AM is at least as good a=
s
> the old.
>=20
> AMv2 is a long-standing effort addressing the most common source of troubl=
e
> operating an hbase cluster at scale, region assignment.
>=20
> AMv2 is the last hurdle to our being able to branch for hbase2.
>=20
> Please vote +1 if in favor of merge and -1 if agin.
>=20
> See below if you need more background on what we're voting on.
>=20
> St.Ack
>=20
> The new AssignmentManager on HBASE-14614 branch promises new levels of
> scaling, insight, resilience and assignment performance. A skeleton dev
> overview is available here [1].
>=20
> There are outstanding items to complete but they can come in post-merge.
> Critical is reenabling a set of tests disabled because they depend on
> workings no longer present or they depended on a broken behavior in AMv1
> (Disabled tests and the list of TODOs are in [1]).
>=20
> Currently I am working on polish making it pass the outstanding failing
> unit tests, fixing white space, and findbugs but it is ready for commit.
> I've put up an RB patch that is absent protobufs so the bulk is less
> intimidating [3].
>=20
> This patch is mostly the work of Matteo Bertozzi (inception, bulk of
> implementation). Others have contributed including Stephen Yuan Jiang.
> Umesh Agashe and Appy also have contributed fixes that got added to the
> HBASE-14614 branch.
>=20
> 1. https://docs.google.com/document/d/1eVKa7FHdeoJ1-
> 9o8yZcOTAQbv0u0bblBlCCzVSIn69g/edit#
> 2. https://docs.google.com/document/d/1QLXlVERKt5EMbx_
> EL3Y2u0j64FN-_TrVoM5WWxIXh6o/edit#heading=3Dh.df9krsl9k16
> 3. https://reviews.apache.org/r/55117/
> 4.
> http://apache-hbase.679495.n3.nabble.com/DISCUSS-Merge-new-Assignment-Mana=
ger-AMv2-HBASE-14614-td4088089.html