-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Substream cursor #28278
Comments
@maxi297 any updates here? This seems to be a blocker for me using airbyte for Chorus's api. I can incrementally pull conversations, but they don't include the transcript. So I need to use a stream partition to query another endpoint for each conversation. However, I haven't found a way to do that where it isn't a full refresh on the substream. |
@tjhiggins I see that some people have shown their interest for this issue. Let me bring this back to the team and see if it's enough to prioritize it. In the meanwhile if this is blocking for you and you are up for the challenge, you could implement your own version of
I'll keep you posted on this issue! |
Grooming:
|
@tjhiggins This has been deemed not aligned with our team's current goals. We will re-evaluate before the next cycle which is around mid-November |
Thanks for the update. |
Another request for this feature: https://airbytehq-team.slack.com/archives/C027KKE4BCZ/p1705077322351399 |
And another one as well from Slack: https://airbytehq.slack.com/archives/C027KKE4BCZ/p1715089626142729 I have one API that would align with this as well, since the child object is only changed when the parent is changed, so a feature like this it would prevent about 100K unneeded requests per run, which would also help with their strict API limits. |
Same here, Jiminny API. |
Any update on this? |
@mariana-s-fernandes @TorstenFraust @NAjustin Can you confirm if this solves your use cases? |
@lmossman What it the behaviour if the substream is not incremental? Right now it looks to me like the substream runs before the parent stream. |
@TorstenFraust have you found a solution or workaround, specifically related to Jiminny? |
Is there any update on the above? @lmossman it has not solved the usecase that @TorstenFraust mentioned. I'm having the exact same problem. Regardless of whether that option is selected, the child still runs a full refresh every time. @htkapiche were you able to solve this at all? |
I have raised the request to support this on non-incremental child streams to the team. I don't have a guarantee on when we will get to it but they are hoping to tackle it soon |
What area the feature impact?
Connectors
Revelant Information
As requested in Slack: For example, the first time I get 3 ids from the API /v1/deals, I pass it to the API /v1/deals/{id}/flow, the second time I run the API /v1/deals, I get 2 new ids, then I pass it to the API /v1 /deals/{id}/flow. How to do this?
As of 2023-07-13, there are no ways to do this because it's a whole new way of managing the state (it's not incremental as "the ids are incremental" but it's incremental as "we have never fetched the information for those ids).
Proposed solution
Have a new component (name to be reworked) to allow the cursor to manage a substream like this:
The text was updated successfully, but these errors were encountered: