You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hbase.apache.org by "张铎(Duo Zhang)" <pa...@gmail.com> on 2021/12/31 04:35:50 UTC
[RESULT][VOTE] Merge feature branch HBASE-26233 back to master
WIth 7 +1 votes from our committers, the vote passes.
I've rebased HBASE-26233 branch against the newest master and
triggered a nightly build. Will merge to master later today if no big
problems.
Thanks all for voting.
Peter Somogyi <ps...@apache.org> 于2021年12月29日周三 23:22写道：
>
> +1 (binding)
>
> Regards,
> Peter
>
> On Wed, Dec 29, 2021 at 7:21 AM 张铎(Duo Zhang) <pa...@gmail.com> wrote:
>
> > Thanks all for voting.
> >
> > Let's wait till this Friday to see if there are more comments.
> >
> > Andrew Purtell <ap...@apache.org> 于2021年12月29日周三 02:27写道：
> > >
> > > +1 (binding)
> > >
> > > Nice work, looking forward to having it land in master.
> > >
> > >
> > > On Sat, Dec 18, 2021 at 12:31 AM Duo Zhang <zh...@apache.org> wrote:
> > >
> > > > There is a previous thread[1] to mention that the main development
> > work on
> > > > HBASE-26233[2] is done. And now the testing work has been done too, so
> > I
> > > > send this official vote to merge for HBASE-26233 back into master.
> > > >
> > > > For those who are not familiar with HBASE-26233, it is for
> > re-implementing
> > > > the region replication framework, to decouple with the general
> > replication
> > > > framework. You could see the design doc[3] for more details. I've also
> > > > opened a PR against master[4], PTAL if you have interest.
> > > >
> > > > You can see the final test result on HBASE-26487[5].
> > > > In conclusion, there is no regression on performance when running PE.
> > > > Using the new tool introduced in HBASE-26540[6], the new
> > implementation can
> > > > perform much better than the old implementation. When inserting 1M
> > rows,
> > > > the max lag on secondary replica is 335ms, and the 99.9% lag is 3 only
> > 3ms,
> > > > while the lag of the old implementation highly depends on the
> > > > replication.source.sleepforretries config but you should not set it to
> > a
> > > > very small value as it will waste a lot of resources.
> > > > On IntegrationTestRegionReplicaReplication, both implementations can
> > not
> > > > pass but the problems are not related to the changing part, so should
> > not
> > > > be a blocker. Will file follow on issues to address these problems.
> > > >
> > > > The vote will open for at least 72 hours.
> > > >
> > > > Please vote:
> > > > +1: Merge the changes from HBASE-26233 to master
> > > > -1: Do not merge these changes because ...
> > > >
> > > > Thanks.
> > > >
> > > > 1. https://lists.apache.org/thread/t8w053kpy9pj1vg1x3c0kjjh4ongrxzy
> > > > 2. https://issues.apache.org/jira/browse/HBASE-26233
> > > > 3.
> > > >
> > > >
> > https://docs.google.com/document/d/1WPSBwRqIPgTrPRM2wtzAe-3XUvA9S7auWy3Fy8xuZLc/edit
> > > > 4. https://github.com/apache/hbase/pull/3891
> > > > 5. https://issues.apache.org/jira/browse/HBASE-26487
> > > > 6. https://issues.apache.org/jira/browse/HBASE-26540
> > > >
> > >
> > >
> > > --
> > > Best regards,
> > > Andrew
> > >
> > > Words like orphans lost among the crosstalk, meaning torn from truth's
> > > decrepit hands
> > > - A23, Crosstalk
> >
Re: [RESULT][VOTE] Merge feature branch HBASE-26233 back to master
Posted by "张铎(Duo Zhang)" <pa...@gmail.com>.
The last build finished successfully.
https://ci-hadoop.apache.org/job/HBase/job/HBase%20Nightly/job/HBASE-26233/15/
Let me merge the feature branch back.
张铎(Duo Zhang) <pa...@gmail.com> 于2021年12月31日周五 12:35写道：
>
> WIth 7 +1 votes from our committers, the vote passes.
>
> I've rebased HBASE-26233 branch against the newest master and
> triggered a nightly build. Will merge to master later today if no big
> problems.
>
> Thanks all for voting.
>
> Peter Somogyi <ps...@apache.org> 于2021年12月29日周三 23:22写道：
> >
> > +1 (binding)
> >
> > Regards,
> > Peter
> >
> > On Wed, Dec 29, 2021 at 7:21 AM 张铎(Duo Zhang) <pa...@gmail.com> wrote:
> >
> > > Thanks all for voting.
> > >
> > > Let's wait till this Friday to see if there are more comments.
> > >
> > > Andrew Purtell <ap...@apache.org> 于2021年12月29日周三 02:27写道：
> > > >
> > > > +1 (binding)
> > > >
> > > > Nice work, looking forward to having it land in master.
> > > >
> > > >
> > > > On Sat, Dec 18, 2021 at 12:31 AM Duo Zhang <zh...@apache.org> wrote:
> > > >
> > > > > There is a previous thread[1] to mention that the main development
> > > work on
> > > > > HBASE-26233[2] is done. And now the testing work has been done too, so
> > > I
> > > > > send this official vote to merge for HBASE-26233 back into master.
> > > > >
> > > > > For those who are not familiar with HBASE-26233, it is for
> > > re-implementing
> > > > > the region replication framework, to decouple with the general
> > > replication
> > > > > framework. You could see the design doc[3] for more details. I've also
> > > > > opened a PR against master[4], PTAL if you have interest.
> > > > >
> > > > > You can see the final test result on HBASE-26487[5].
> > > > > In conclusion, there is no regression on performance when running PE.
> > > > > Using the new tool introduced in HBASE-26540[6], the new
> > > implementation can
> > > > > perform much better than the old implementation. When inserting 1M
> > > rows,
> > > > > the max lag on secondary replica is 335ms, and the 99.9% lag is 3 only
> > > 3ms,
> > > > > while the lag of the old implementation highly depends on the
> > > > > replication.source.sleepforretries config but you should not set it to
> > > a
> > > > > very small value as it will waste a lot of resources.
> > > > > On IntegrationTestRegionReplicaReplication, both implementations can
> > > not
> > > > > pass but the problems are not related to the changing part, so should
> > > not
> > > > > be a blocker. Will file follow on issues to address these problems.
> > > > >
> > > > > The vote will open for at least 72 hours.
> > > > >
> > > > > Please vote:
> > > > > +1: Merge the changes from HBASE-26233 to master
> > > > > -1: Do not merge these changes because ...
> > > > >
> > > > > Thanks.
> > > > >
> > > > > 1. https://lists.apache.org/thread/t8w053kpy9pj1vg1x3c0kjjh4ongrxzy
> > > > > 2. https://issues.apache.org/jira/browse/HBASE-26233
> > > > > 3.
> > > > >
> > > > >
> > > https://docs.google.com/document/d/1WPSBwRqIPgTrPRM2wtzAe-3XUvA9S7auWy3Fy8xuZLc/edit
> > > > > 4. https://github.com/apache/hbase/pull/3891
> > > > > 5. https://issues.apache.org/jira/browse/HBASE-26487
> > > > > 6. https://issues.apache.org/jira/browse/HBASE-26540
> > > > >
> > > >
> > > >
> > > > --
> > > > Best regards,
> > > > Andrew
> > > >
> > > > Words like orphans lost among the crosstalk, meaning torn from truth's
> > > > decrepit hands
> > > > - A23, Crosstalk
> > >