[
https://issues.apache.org/jira/browse/HBASE-13121?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14351157#comment-14351157
]
Enis Soztutar commented on HBASE-13121:
---------------------------------------
Thanks Jeffrey for review.
bq. 1) Could you still set recovering to false and then submit the rest work
into executor
I thought about this. I think it is safer to do it this way. The expectation is
that there will be executors available.
bq. 2) openSeqNum in the following code may still use the old value?
Good catch. I've modified it so that we get a new seqId and use it as the open
seqId. Since during replay, we can accept more writes, the seqId maybe bigger
than the original open seq id.
> Async wal replication for region replicas and dist log replay does not work
> together
> ------------------------------------------------------------------------------------
>
> Key: HBASE-13121
> URL: https://issues.apache.org/jira/browse/HBASE-13121
> Project: HBase
> Issue Type: Sub-task
> Reporter: Enis Soztutar
> Assignee: Enis Soztutar
> Fix For: 2.0.0, 1.1.0
>
> Attachments: hbase-13121_v1.patch
>
>
> We had not tested dist log replay while testing async wal replication for
> region replicas. There seems to be a couple of issues, but fixable.
> The distinction for dist log replay is that, the region will be opened for
> recovery and regular writes when a primary fails over. This causes the region
> open event marker to be written to WAL, but at this time, the region actually
> does not contain all the edits flushed (since it is still recovering). If
> secondary regions see this event, and picks up all the files in the region
> open event marker, then they can drop edits.
> The solution is:
> - Only write the region open event marker to WAL when region is out of
> recovering mode.
> - Force a flush out of recovering mode. This ensures that all data is force
> flushed in this case. Before the region open event marker is written, we
> guarantee that all data in the region is flushed, so the list of files in the
> event marker is complete.
> - Edits coming from recovery are re-written to WAL when recovery is in
> action. These edits will have a larger seqId then their "original" seqId. If
> this is the case, we do not replicate these edits to the secondary replicas.
> Since the dist log replay recovers edits out of order (coming from parallel
> replays from WAL file split tasks), this ensures that TIMELINE consistency is
> respected and edits are not seen out of order in secondaries. These edits are
> seen from secondaries via the forced flush event.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
