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

extract-jdbc: cursor incremental query includes lb #44450

Closed

Conversation

postamar
Copy link
Contributor

@postamar postamar commented Aug 20, 2024

The cursor's lower bound value needs to be included in the WHERE clause because the cursor column values may be not unique; right now we're facing data loss for example if the cursor column is of type DATE DEFAULT NOW() and rows get inserted after a daily sync but on the same day.

Copy link

vercel bot commented Aug 20, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
airbyte-docs ⬜️ Ignored (Inspect) Visit Preview Aug 20, 2024 8:56pm

@octavia-squidington-iii octavia-squidington-iii added the CDK Connector Development Kit label Aug 20, 2024
Copy link
Contributor Author

postamar commented Aug 20, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @postamar and the rest of your teammates on Graphite Graphite

@postamar postamar force-pushed the postamar_gt_08-20-extract-jdbc_add_greaterorequal branch from efdfacf to 6c36470 Compare August 20, 2024 20:56
@postamar
Copy link
Contributor Author

Closing because I'm merging the top of the stack #44482 into master using slash-approve-and-merge.

@postamar postamar closed this Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CDK Connector Development Kit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants