-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
cdc: Data may be lost during leader transfer #193
Comments
cc@pingyu @haojinming |
Some additional information:
We may conclude that the node |
* resolvedts - x Signed-off-by: zeminzhou <[email protected]> * resolvedts - x Signed-off-by: zeminzhou <[email protected]> * resolvedts - x Signed-off-by: zeminzhou <[email protected]> * fix comment & ut Signed-off-by: zeminzhou <[email protected]> * add ut Signed-off-by: zeminzhou <[email protected]> * fix ut Signed-off-by: zeminzhou <[email protected]> * fix ut Signed-off-by: zeminzhou <[email protected]> * fix comment Signed-off-by: zeminzhou <[email protected]> * fix ut Signed-off-by: zeminzhou <[email protected]> * remove tmp Signed-off-by: zeminzhou <[email protected]> * fix comment Signed-off-by: zeminzhou <[email protected]> * fix ut Signed-off-by: zeminzhou <[email protected]> * fix kv ut timeout Signed-off-by: zeminzhou <[email protected]> * fix ut Signed-off-by: zeminzhou <[email protected]> * fix check Signed-off-by: zeminzhou <[email protected]> * fix ut Signed-off-by: zeminzhou <[email protected]>
Bug Report
1. Describe the bug
In TiKV, when a leader transfer occurs, the old leader may send the last resolved ts, which may be larger than the new leader's first causal timestamp after transfer.
Details as follow(from log):
So B don't send the key that is with ts(435080671018356350) to cdc.
2. Minimal reproduce step (Required)
3. What did you see instead (Required)
4. What did you expect to see? (Required)
The text was updated successfully, but these errors were encountered: