-
Notifications
You must be signed in to change notification settings - Fork 4.6k
✨ Source Zendesk Support: ticket metrics streams to low-code #55795
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
base: issue-11894/article_streams-to-low-code
Are you sure you want to change the base?
✨ Source Zendesk Support: ticket metrics streams to low-code #55795
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Note that part of the CI has been disabled which prevents me from running regression tests. I'll be doing this later |
airbyte-integrations/connectors/source-zendesk-support/source_zendesk_support/source.py
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this was a clever migration and very cool to see it done w/o custom components.
You mention this is breaking, does that mean we need to bump major?
What
Partially addresses https://github.com/airbytehq/airbyte-internal-issues/issues/11894
Migrating ticket_metrics to low-code.
There are breaking change: the state of the stateful implementation is changing for a per partition state.
How
Update the CDK version and using the new state delegating stream.
Review guide
User Impact
This change is mostly to ease maintenance for devs and there shouldn't be any change in behavior.
Can this PR be safely reverted and rolled back?
The state format is changing hence we can revert this change.