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 <psomo...@apache.org> 于2021年12月29日周三 23:22写道： > > +1 (binding) > > Regards, > Peter > > On Wed, Dec 29, 2021 at 7:21 AM 张铎(Duo Zhang) <palomino...@gmail.com> wrote: > > > Thanks all for voting. > > > > Let's wait till this Friday to see if there are more comments. > > > > Andrew Purtell <apurt...@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 <zhang...@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 > >
