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

bulk-cdk-core-extract: rewrite commentary for MetadataQuerier #44453

Conversation

postamar
Copy link
Contributor

@postamar postamar commented Aug 20, 2024

The current DiscoverOperation is overfitted to SQL database schemas and takes a bunch of unpleasant shortcuts which won't age well. This PR addresses some of this.

Informs airbytehq/airbyte-internal-issues#9093

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
@postamar postamar marked this pull request as ready for review August 20, 2024 15:34
@postamar postamar requested a review from a team as a August 20, 2024 15:34
@postamar postamar force-pushed the postamar_gt_08-20-extract-jdbc_better_handling_of_null_cursor_values branch from 51279be to 639f09c Compare August 20, 2024 15:44
@postamar postamar force-pushed the postamar_gt_08-20-bulk-cdk-core-extract_rewrite_commentary_for_metadataquerier branch from 6390f94 to bb3ea8e Compare August 20, 2024 15:44
@postamar postamar force-pushed the postamar_gt_08-20-extract-jdbc_better_handling_of_null_cursor_values branch from 639f09c to 35ce6ae Compare August 20, 2024 18:47
@postamar postamar force-pushed the postamar_gt_08-20-bulk-cdk-core-extract_rewrite_commentary_for_metadataquerier branch from bb3ea8e to fa4ccf4 Compare August 20, 2024 18:47
@postamar postamar force-pushed the postamar_gt_08-20-extract-jdbc_better_handling_of_null_cursor_values branch from 35ce6ae to 3131abd Compare August 20, 2024 20:56
@postamar postamar force-pushed the postamar_gt_08-20-bulk-cdk-core-extract_rewrite_commentary_for_metadataquerier branch from fa4ccf4 to 53e69bb 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