HBASE-27516 Document the table based replication queue storage in ref guide #5203
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
Apache9
reviewed
Apr 26, 2023
| By default, the state is contained in the base node _/hbase/replication_. | ||
| Usually this nodes contains two child nodes, the `peers` znode is for storing replication peer | ||
| state, and the `rs` znodes is for storing replication queue state. | ||
| Currently, this nodes contains only one child node, namely `peers` znode, which is used for storing replication peer state. |
Contributor
There was a problem hiding this comment.
As now we only have one ref guide on master branch, we should include the doducmentation for all branches. So here I think we should mention that, after 3.0.0, it only contains one child node, but before 3.0.0, we still use zk to store queue data.
| Each master cluster region server has its own znode in the replication znodes hierarchy. | ||
| It contains one znode per peer cluster (if 5 slave clusters, 5 znodes are created), and each of these contain a queue of WALs to process. | ||
| Each master cluster region server has its queue state in the hbase:replication table. | ||
| It contains one row per peer cluster (if 5 slave clusters, 5 rows are created), and each of these contain a queue of WALs to process. |
Contributor
There was a problem hiding this comment.
Here things are a bit different. For zookeeper, it is like a tree, we have a znode for a peer cluster, but under the znode we have lots of files.
But for table based implementation, we have server name in row key, which means we will have lots of rows for a given peer...
| After queues are all transferred, they are deleted from the old location. | ||
| The znodes that were recovered are renamed with the ID of the slave cluster appended with the name of the dead server. | ||
| When a region server fails, the HMaster of master cluster will trigger the SCP, and all replication queues on the failed region server will be claimed in the SCP. | ||
| The claim queue operation is just to remove the row of a replication queue, and insert a new row, where we change the server name to the region server which claims the queue. |
Contributor
There was a problem hiding this comment.
We need to mention that we use multi row mutate endpoint here, so the data for a single peer must be in the same region.
|
🎊 +1 overall
This message was automatically generated. |
Apache9
force-pushed
the
HBASE-27109/table_based_rqs
branch
from
b5535c9 to
b1a7069
Compare
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
Apache9
reviewed
Apr 27, 2023
| The value of each peer is its cluster key, which is provided in the HBase Shell. | ||
| The cluster key contains a list of ZooKeeper nodes in the cluster's quorum, the client port for the ZooKeeper quorum, and the base znode for HBase in HDFS on that cluster. | ||
|
|
||
| The `RS` Znode:: |
Contributor
There was a problem hiding this comment.
We'd better keep this unchanged, as it describes what we have before 3.0.0. And we can introduce a new section to describe the hbase:replication table storage.
| By default, the state is contained in the base node _/hbase/replication_. | ||
| Usually this nodes contains two child nodes, the `peers` znode is for storing replication peer | ||
| state, and the `rs` znodes is for storing replication queue state. | ||
| After 3.0.0, it only contains one child node, but before 3.0.0, we still use zk to store queue data. |
Contributor
There was a problem hiding this comment.
"Usually this nodes contains two child nodes, the peers znode is for storing replication peer state, and the rs znodes is for storing replication queue state. And if you choose the file system based replication peer storage, you will not see the peers znode. And starting from 3.0.0, we have moved the replication queue state to hbase:replication table, so you will not see the rs znode."
| states. | ||
| HBASE-15867 is only half done, as although we have abstract these two interfaces, we still only | ||
| have zookeeper based implementations. | ||
| And in HBASE-27109, we have implemented the `ReplicationQueueStorage` interface to store the replication queue in the hbase:replication table. |
Contributor
There was a problem hiding this comment.
And in HBASE-27110, we have implemented a file system based replication peer storage, to store replication peer state on file system. Of course you can still use the zookeeper based replication peer storage.
And in HBASE-27109, we have changed the replication queue storage from zookeeper based to hbase table based. See the below 'Replication Queue State in hbase:replication table' section for more details.
|
|
||
| Each master cluster region server has its own znode in the replication znodes hierarchy. | ||
| It contains one znode per peer cluster (if 5 slave clusters, 5 znodes are created), and each of these contain a queue of WALs to process. | ||
| Before 3.0.0, for zookeeper based implementation, it is like a tree, we have a znode for a peer cluster, but under the znode we have lots of files. |
Contributor
There was a problem hiding this comment.
I think here we'd better make two different sections to describe the logic before and after 3.0.0. As on zookeeper, we store all WAL files on it and for table based solution, we only store an offset.
|
|
||
| When a source is instantiated, it contains the current WAL that the region server is writing to. | ||
| During log rolling, the new file is added to the queue of each slave cluster's znode just before it is made available. | ||
| During log rolling, the new file is added to the queue of each slave cluster's record just before it is made available. |
Contributor
There was a problem hiding this comment.
This is different for table based replication queue storage, and it is the key point here. For zookeeper, it is an external system so there is no problem to let log rolling depend on it, but if we want to store the state in a hbase table, we can not let log rolling depend on it as it will introduce dead lock...
We will only write to hbase:replication when want to record an offset after replicating something.
| When no region servers are failing, keeping track of the logs in ZooKeeper adds no value. | ||
| Unfortunately, region servers do fail, and since ZooKeeper is highly available, it is useful for managing the transfer of the queues in the event of a failure. | ||
|
|
||
| Each of the master cluster region servers keeps a watcher on every other region server, in order to be notified when one dies (just as the master does). When a failure happens, they all race to create a znode called `lock` inside the dead region server's znode that contains its queues. |
Contributor
There was a problem hiding this comment.
I think we still need to keep this, as it is the logic for some hbase releases. Let me check the version where we start to use SCP to claim replication queue.
Apache9
force-pushed
the
HBASE-27109/table_based_rqs
branch
3 times, most recently
from
017a9d3 to
772acaa
Compare
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
Contributor
Author
|
@Apache9 sir. Could you take a look? Thanks. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
Apache9
reviewed
May 5, 2023
| This list includes both live and dead region servers. | ||
|
|
||
| [[hbase:replication]] | ||
| hbase:replication:: |
Contributor
There was a problem hiding this comment.
Use "The hbase:replication Table"?
|
|
||
| After 3.0.0, for table based implementation, we have server name in row key, which means we will have lots of rows for a given peer. | ||
|
|
||
| For a normal replication queue, where the WAL files belong to it is still alive, all the WAL files are kept in memory, so we do not need to get the WAL files from replication queue storage. |
Contributor
There was a problem hiding this comment.
"the region server is still alive"
|
|
||
| [[rs.failover.details]] | ||
| ==== Region Server Failover | ||
| ==== Region Server Failover(based on ZooKeeper) |
Contributor
There was a problem hiding this comment.
Here we do not need to se two different sections. First, we could mention the 'setting a watcher' way, this is way in the old time.
And starting from 2.5.0, the failover logic has been moved to SCP, where we add a SERVER_CRASH_CLAIM_REPLICATION_QUEUES step in SCP to claim the replication queues for a dead server.
And starting from 3.0.0, where we changed the replication queue storage from zookeeper to table, the update to the replication queue storage is async, so we also need an extra step to add the missing replication queues before claiming.
And on how to claim the replication queue, you can have two sections, to describe the layout and claiming way for zookeeper based implementation and table based implementation.
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
Apache9
approved these changes
May 6, 2023
Contributor
Apache9
left a comment
Apache9
left a comment
There was a problem hiding this comment.
No big concerns frome me, just a minor nit.
Thanks @2005hithlj !
| Assume that 1.1.1.2 failed. | ||
| The survivors will claim queue of that, and, arbitrarily, 1.1.1.3 wins. | ||
| It will claim all the queue of 1.1.1.2, including removing the row of a replication queue, and inserting a new row(where we change the server name to the region server which claims the queue). | ||
| Finally ,the layout will look like the following: |
Contributor
There was a problem hiding this comment.
The position of the ',' is incorrect?
Contributor
Author
There was a problem hiding this comment.
The position of the ',' is incorrect?
OK sir , I have revised.
Apache9
approved these changes
May 6, 2023
Contributor
Apache9
left a comment
Apache9
left a comment
There was a problem hiding this comment.
No big concerns frome me, just a minor nit.
Thanks @2005hithlj !
Apache9
approved these changes
May 6, 2023
Contributor
Apache9
left a comment
Apache9
left a comment
There was a problem hiding this comment.
No big concerns frome me, just a minor nit.
Thanks @2005hithlj !
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
Apache9
pushed a commit
that referenced
this pull request
May 6, 2023
… guide (#5203) Signed-off-by: Duo Zhang <[email protected]>
Apache9
pushed a commit
that referenced
this pull request
May 8, 2023
… guide (#5203) Signed-off-by: Duo Zhang <[email protected]>
Apache9
pushed a commit
that referenced
this pull request
May 13, 2023
… guide (#5203) Signed-off-by: Duo Zhang <[email protected]>
Apache9
pushed a commit
that referenced
this pull request
May 14, 2023
… guide (#5203) Signed-off-by: Duo Zhang <[email protected]>
Apache9
pushed a commit
that referenced
this pull request
May 15, 2023
… guide (#5203) Signed-off-by: Duo Zhang <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
https://issues.apache.org/jira/browse/HBASE-27516