Skip to content
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

Adding stream support to Dynalite #70

Closed
wants to merge 1 commit into from

Conversation

mcwhittemore
Copy link

@mhart - I need stream support for some tests I'm writing. Is this something you are interested in adding to dynalite? I see its mentioned in the TODOs.

Todos for this PR:

  • only allow streams for tables with stream configs
  • send old and new items per the stream config
  • support batch sizes
  • valid event schema (right now it's just the subsections I need)

/cc @rclark @scothis

@mhart
Copy link
Collaborator

mhart commented Jan 3, 2017

@mcwhittemore thanks for the PR! I gather you haven't seen #67 😸

@mcwhittemore
Copy link
Author

Oh man. Sorry. I looked for a PR a few weeks ago. Nice to see this is moving.

@mhart
Copy link
Collaborator

mhart commented Jan 3, 2017

No worries – now that the holidays are over I should have some time to look at how best to achieve this – so perhaps things can be pulled together from both your work and that of @kiril-pirozenko-home24

@mcwhittemore
Copy link
Author

@kiril-pirozenko-home24's is much closer to be done it looks like. Though, mine was designed around hooking up triggers and I'm not 100% sure how to do that with what's in #67 right now. I'll close this and make some comments over there.

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

Successfully merging this pull request may close these issues.

2 participants