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

Give dispute stream a 90-day lookback window. #6

Merged
merged 1 commit into from
Jan 25, 2024

Conversation

omarabed15
Copy link
Collaborator

@omarabed15 omarabed15 commented Jan 24, 2024

This PR increases the lookback window for the disputes stream to 90 days, from 1 day.
This was done because we are fetching dispute objects, not events, from the Stripe stream, so we are not capturing updates to the dispute objects by using a 1-day lookback. By increasing the lookback window, we will capture updates to any dispute objects created in the last 90 days.

Tests

Executed the tap through the meltano project with:
meltano --environment=dev elt tap-stripe target-jsonl --job_id=elt-stripe-to-json --select disputes

Saw results going back to Friday, October 27, 2023 11:13:33 PM in the JSON output.

Execution took under 25 seconds:

image ... image

@omarabed15 omarabed15 force-pushed the DT-3007-disputes-incremental-logic branch from 4d06194 to a5ebe06 Compare January 24, 2024 16:59
@omarabed15 omarabed15 merged commit 722f445 into main Jan 25, 2024
0 of 3 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