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

feat: Use sorted API requests to support crash tolerant incremental sync #32

Merged
merged 5 commits into from
Dec 15, 2023

Conversation

JichaoS
Copy link
Contributor

@JichaoS JichaoS commented Dec 14, 2023

The streams changed support sorted responses

Ex: https://developers.klaviyo.com/en/reference/get_campaigns

This PR takes advantage of ascending timestamps to support fault tolerant incremental syncs that picks up at the last left-off place.

@JichaoS JichaoS changed the title Feat: Use sorted API requests to support crash tolerant incremental sync feat: Use sorted API requests to support crash tolerant incremental sync Dec 14, 2023
@edgarrmondragon
Copy link
Member

Hi @JichaoS! Thanks for contributing. Can you merge main into your branch?

@JichaoS
Copy link
Contributor Author

JichaoS commented Dec 14, 2023

@edgarrmondragon Updated!

@JichaoS
Copy link
Contributor Author

JichaoS commented Dec 14, 2023

@edgarrmondragon which API version is the secret pointing to? Can we update that to at min "2023-02-22" (which is what I've been running with)

@edgarrmondragon
Copy link
Member

@JichaoS It's pointing to revision 2023-10-15, but you probably need to set the secrets in your fork for GH actions to pick them up.

@JichaoS
Copy link
Contributor Author

JichaoS commented Dec 15, 2023

Thanks, I tested this out locally and it's WAI. Getting same results as meltanolabs/head on my test acct.

Copy link
Member

@edgarrmondragon edgarrmondragon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @JichaoS

@edgarrmondragon edgarrmondragon merged commit b723520 into MeltanoLabs:main Dec 15, 2023
0 of 5 checks passed
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