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

Destinations bigquery+snowflake+redshift: hard revert refreshes thing #44526

Merged
merged 5 commits into from
Aug 22, 2024

Conversation

edgao
Copy link
Contributor

@edgao edgao commented Aug 21, 2024

stop querying the db, just return 0 for generation ID instead. This means that during truncate syncs, we will always assume the final table belongs to the wrong generation (i.e. equivalent to behavior from before https://github.com/airbytehq/airbyte-internal-issues/issues/8784). As a reminder, platform has a mitigation for this bug (https://github.com/airbytehq/airbyte-platform-internal/pull/13190 - only send the success status message if the source exits 0)

Also disable the relevant test case for these destinations.

Copy link

vercel bot commented Aug 21, 2024

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 Aug 22, 2024 4:56pm

@edgao edgao force-pushed the edgao/hard_revert_refreshes_stuff branch from 5c94292 to 5b4fa6a Compare August 22, 2024 16:51
@octavia-squidington-iii octavia-squidington-iii added the area/documentation Improvements or additions to documentation label Aug 22, 2024
@edgao edgao merged commit ee532e2 into master Aug 22, 2024
39 checks passed
@edgao edgao deleted the edgao/hard_revert_refreshes_stuff branch August 22, 2024 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants