+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 >