-
Notifications
You must be signed in to change notification settings - Fork 2
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
Use VStream() #120
Merged
Merged
Use VStream() #120
Changes from all commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
b1853c3
Adding VStream
notfelineit f885974
Split out
notfelineit 7a6e7a4
Add vtgate service
notfelineit 309f337
Update proto/vtgateservice.proto
notfelineit b2120e3
Update Makefile
notfelineit 3f0fede
Run tidy
notfelineit 21918b2
Merge branch 'use-vstream' of github.com:planetscale/airbyte-source i…
notfelineit c47c820
Try modifying getLatestCursorPosition to use vtgate
notfelineit d77e959
Reset protos
notfelineit 06f5d6d
Get getLatestCursor working locally
notfelineit 00a549f
Get local sync working
notfelineit 772b9ae
Undo changes to makefile
notfelineit a873826
Refactor and reuse some logic, add helpful logs
notfelineit 0baf971
Clean up logs more
notfelineit 3c744d6
re run proto
notfelineit 5b061ec
Update mock types and modify first few tests
notfelineit 2238dcc
Convert more tests
notfelineit 45f45d5
Handle cases of copy phase and first VGTID is past stop position, and…
notfelineit 6c4e381
Remove some noisy logs
notfelineit 4b8c976
Update test testing stopping at next VGtid after stop position
notfelineit 375dbc8
Finish tests
notfelineit 317637e
Remove unused mock types
notfelineit 2c22667
Remove unused method
notfelineit 3fffdc9
Simplify var names and logic
notfelineit 0a6aed6
Clean up some more logs
notfelineit 2f4559e
Handle case of no new VGTID found
notfelineit 7026de9
Remove unused method
notfelineit cbf56ba
Remove extra spaces
notfelineit 787467c
Address some PR comments
notfelineit 02057d4
Clean up protos
notfelineit 6689561
Add test for copy catchup
notfelineit File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
Oops, something went wrong.
Oops, something went wrong.
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.
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.
a little unclear to me what
lastResponseSent
is tracking.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.
So I am pretty sure last response sent keeps track of which index of mock sync responses should be sent next!