+1 from me.

Thanks.

在 2021/12/27 12:20,“张铎  (Duo Zhang)”<palomino...@gmail.com> 写入:

    We still need more votes here for merging the feature branch back.

    As this is a code change, the committee's vote will also be counted,
    not only the PMC member's vote will be counted.

    Thanks.

    Guanghao Zhang <zghao...@gmail.com> 于2021年12月24日周五 18:02写道:
    >
    > +1 from me.
    >
    > The idea and solution is great!
    >
    > Thanks.
    >
    > Duo Zhang <zhang...@apache.org> 于2021年12月18日周六 16:31写道:
    >
    > > 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
    > >


Reply via email to