Mailing-List: contact dev-help@hbase.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@hbase.apache.org
MIME-Version: 1.0
Sender: saint.ack@gmail.com
In-Reply-To: <CADcMMgG7mOHMV52D5BGrWHLY6tp-z=+wT9uvGOPdyQEcskbGQA@mail.gmail.com>
References: <CADcMMgG7mOHMV52D5BGrWHLY6tp-z=+wT9uvGOPdyQEcskbGQA@mail.gmail.com>
From: Stack <stack@duboce.net>
Date: Wed, 31 May 2017 16:24:31 -0700
Message-ID: <CADcMMgFv7YRocMQbK+jyygziD8_OqsC=pbQgieezWbN2-F4BPg@mail.gmail.com>
Subject: Re: VOTE: Merge new Assignment Manager AMv2), HBASE-14614
To: HBase Dev List <dev@hbase.apache.org>
Content-Type: multipart/alternative; boundary="001a114e532af0b7eb0550da38c9"
archived-at: Wed, 31 May 2017 23:24:37 -0000

--001a114e532af0b7eb0550da38c9
Content-Type: text/plain; charset="UTF-8"

Thanks all for the votes.

I'll go ahead and merge this evening. I'd imagine there'll be some teething
issues given size of the patch but smoothing them over will be my full time
job here on out. Thanks in advance for you patience!

St.Ack


On Tue, May 30, 2017 at 9:09 AM, Stack <stack@duboce.net> wrote:

> Moving on from the DISCUSSION thread [4], lets vote on whether to merge
> the new Assignment Manager (AMv2) out on branch HBASE-14614 to master.
>
> Unit tests pass. Tests at scale show that the new AM is at least as good
> as the old.
>
> AMv2 is a long-standing effort addressing the most common source of
> trouble operating an hbase cluster at scale, region assignment.
>
> AMv2 is the last hurdle to our being able to branch for hbase2.
>
> Please vote +1 if in favor of merge and -1 if agin.
>
> See below if you need more background on what we're voting on.
>
> St.Ack
>
> 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].
>
> 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]).
>
> 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].
>
> 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.
>
> 1. https://docs.google.com/document/d/1eVKa7FHdeoJ1-9o8yZcOT
> AQbv0u0bblBlCCzVSIn69g/edit#
> 2. https://docs.google.com/document/d/1QLXlVERKt5EMbx_EL3Y2u
> 0j64FN-_TrVoM5WWxIXh6o/edit#heading=h.df9krsl9k16
> 3. https://reviews.apache.org/r/55117/
> 4. http://apache-hbase.679495.n3.nabble.com/DISCUSS-Merge-
> new-Assignment-Manager-AMv2-HBASE-14614-td4088089.html
>

--001a114e532af0b7eb0550da38c9--