Skip to content

Spanner support #1315

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

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Spanner support #1315

wants to merge 8 commits into from

Conversation

adkhare
Copy link

@adkhare adkhare commented Aug 16, 2019

We at Mailchimp are wanting some feature around this. Purpose is to have a test data replicated over to Spanner so that we can use it for future

@osheroff
Copy link
Collaborator

It's really cool that you guys are doing this. maxwell -> something else that I didn't ever anticipate is great. However, I'm not sure if doing a maxwell producer around this is correct:

  • how would one guarantee that the spanner schema is correctly synchronized with the maxwell schema?
  • what does it do if we find an out of sync schema?
  • maxwell really only provides at-least-once guarantees, duplicate rows here are a possible issue.
  • is one-row-at-a-time going to be fast enough or would we really need some sort of batching?

i mean i guess if the attitude is "whatever. the code works, let it crash" I'm sort of ok with that. I can't really see wholeheartedly recommending folks do this this way though, seems like what you really want is for maxwell to output to an intermediate buffer and then do this in application logic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants