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.
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
TiCDC supports a snapshot-map in replication #932
TiCDC supports a snapshot-map in replication #932
Changes from 13 commits
4e8689e
88318c3
e4cadbd
f26eead
3fd1b3d
97a1615
94ec1d4
6abad54
c13ccc3
abd124b
091927b
0a83609
1da2a73
8f25e2f
b1dad1d
7e2ca72
22fca2a
85d1482
2563a39
17de492
8a16cbd
fb8e6cc
a90718e
aeb7eb5
872c029
09f5043
a15dcec
7cfa8fe
4264afa
97099c6
91bfda1
3c846af
4753199
f6f3993
6ab63c7
416baee
5045d09
3383eab
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I should use package
sink
in L360 (var syncpointStore sink.SyncpointStore
) to create asyncpointStore
,if the var named sink,the packagesink
will not be used in the follow code,there is a conflict.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe move the
sink.SyncpointStore
to another package is better, but it reuses some code in the sink package.We can refine this in another PR.