+1 On Tue, Nov 17, 2020 at 9:49 AM Huaxiang Sun <huaxiang...@gmail.com> wrote:
> +1 > > On Tue, Nov 17, 2020 at 9:21 AM Bharath Vissapragada <bhara...@apache.org> > wrote: > > > +1. Reviewed the design doc and the consolidated patch, great > improvement, > > thanks for putting this together. > > > > On Tue, Nov 17, 2020 at 9:09 AM Stack <st...@duboce.net> wrote: > > > > > +1 > > > S > > > > > > On Tue, Nov 17, 2020 at 8:43 AM Stack <st...@duboce.net> wrote: > > > > > > > Please VOTE on whether to merge HBASE-18070 feature branch to master > > (and > > > > HBASE-18070.branch-2 to branch-2). The VOTE runs for 24 hours. The > > > majority > > > > prevails (+ or -). > > > > > > > > Quoting the design lead-in: > > > > > > > > Read Replicas on the hbase:meta Table currently only does primitive > > read > > > > of the primary’s hfiles refreshing every (configurable) N seconds. > This > > > > issue is about making it so we can do the Async WAL Replication > > > > <http://hbase.apache.org/book.html#_asnyc_wal_replication> ability, > > > > currently only available for user-space Tables, against the > hbase:meta > > > > system Tables too; i.e. the primary replica pushes edits to its > > Replicas > > > so > > > > they run much closer to the primaries’ state. If clients could be > > > satisfied > > > > reading from Replicas, then we could have improved hbase:meta uptimes > > but > > > > also, we can distribute load off of the primary and alleviate > > hbase:meta > > > > Table (read) hotspotting. > > > > > > > > Each PR that comprises the feature branch has been reviewed before > > > commit. > > > > > > > > * For the design, see [2]. > > > > * For an amalgamated PR of the 5 or 6 reviewed PRs that comprise > this > > > > feature, see [3]. > > > > * For a PE report that compared performance before and after, see > > > > HBASE-25127 (no regression). > > > > * A report on ITBLL runs is pending to be attached to HBASE-18070 > but > > > > runs so far show no regression with the feature enabled (ITBLL runs > > were > > > > done against a backport of this feature to branch-2 as the ITBLL > state > > of > > > > master is currently an unknown). > > > > > > > > Testing continues mainly looking for further improvement and to > better > > > > understand this feature in operation. Documentation is included. > There > > > are > > > > some follow-ons that have been identified but these can land later. > > > > > > > > Thanks and thanks to all who contributed to this feature; the > reviewers > > > > and the testers in particular. > > > > > > > > S > > > > > > > > 1. http://hbase.apache.org/book.html#_asnyc_wal_replication > > > > 2. > > > > > > > > > > https://docs.google.com/document/d/1jJWVc-idHhhgL4KDRpjMsQJKCl_NRaCLGiH3Wqwd3O8/edit# > > > > This patch is currently missing HBASE-25280, a bug found in testing. > > > > 3. https://github.com/apache/hbase/pull/2643 > > > > > > > > > >
