-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
✨ Source Tiktok Marketing: Support concurrency #51584
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@tolik0 wrapping this up this week, or are we overflowing into q1? |
@natikgadzhi Yes, I will merge this PR this week |
/format-fix
|
…ent' into maxi297/tiktok-marketing-concurrent
/autofix EDIT: I tried :( |
/format-fix
|
Was this not already the case? Looking at the metabase dashboard, we have a good number of
Are we going to coordinate this release w/ support so they are aware of things? Also, I thought the progressive rollout release process was not suitable for "Progressive rollouts for user-facing changes (which require an opt-in) are not supported yet" |
@brianjlai We don’t have any users in the Cloud who have this option in their config. This parameter was not included in the spec, but we were still renaming the streams, which led to an error. |
understood. thanks for the explanation. no other issues for me |
What
Updated the Airbyte CDK to version 6, which introduces concurrency.
How
The following streams are now concurrent:
User Impact
Syncs should run faster due to improved concurrency settings.
report_granularity
:report_granularity
option has been deprecated.Affected Streams (for users with
report_granularity
):Users must reset their source configuration, refresh the schema, and clear data for the impacted streams after upgrading.
Can this PR be safely reverted and rolled back?
report_granularity
option is present in the config)