+1 for merge On Tue, May 30, 2017 at 10:33 PM, ramkrishna vasudevan < ramkrishna.s.vasude...@gmail.com> wrote:
> I have been following this activity and I think it is good to be merged. > +1. Any test case failures can be fixed post merge (if any). > > Regards > Ram > > On Wed, May 31, 2017 at 3:03 AM, Stephen Jiang <syuanjiang...@gmail.com> > wrote: > > > +1 for merge > > > > On Tue, May 30, 2017 at 2:05 PM, Umesh Agashe <uaga...@cloudera.com> > > wrote: > > > > > +1 for merge. I have reviewed most of this work. > > > > > > On Tue, May 30, 2017 at 10:13 AM, Andrew Purtell < > > andrew.purt...@gmail.com > > > > > > > wrote: > > > > > > > +1 for merge after all unit tests are passing. No problem to exclude > > > known > > > > broken tests not working independent of this patch-set. > > > > > > > > > > > > > On May 30, 2017, at 9:09 AM, Stack <st...@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- > > > > > 9o8yZcOTAQbv0u0bblBlCCzVSIn69g/edit# > > > > > 2. https://docs.google.com/document/d/1QLXlVERKt5EMbx_ > > > > > EL3Y2u0j64FN-_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 > > > > > > > > > >