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(source-klaviyo): Add API Budget #53223

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

Conversation

tolik0
Copy link
Contributor

@tolik0 tolik0 commented Feb 7, 2025

What

This PR introduces API budget tracking.
It helps manage and optimize API usage to prevent exceeding rate limits.

How

  • Updated CDK version to leverage new features and improvements.
  • Added API Budget tracking, allowing better control over API request consumption.
  • Increased default concurrent workers to 50, improving sync speed.

Review guide

User Impact

Syncs should become faster due to the increased default concurrency.

Can this PR be safely reverted and rolled back?

  • YES 💚
  • NO ❌

Copy link

vercel bot commented Feb 7, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
airbyte-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 11, 2025 6:11pm

@tolik0
Copy link
Contributor Author

tolik0 commented Feb 10, 2025

/format-fix

Format-fix job started... Check job output.

✅ Changes applied successfully. (f4db8a3)

@tolik0
Copy link
Contributor Author

tolik0 commented Feb 12, 2025

Removing custom record filter is blocked by: airbytehq/airbyte-python-cdk#330

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/connectors Connector related issues area/documentation Improvements or additions to documentation connectors/source/klaviyo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants