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: rename PartitionsCreator and PartitionReader impls #44457

Conversation

postamar
Copy link
Contributor

@postamar postamar commented Aug 20, 2024

Replaces #44398 (in part)

This is a purely mechanical change that renames StreamPartitionsCreator into JdbcPartitionsCreator and StreamPartitionReader into JdbcPartitionReader. This makes the next PR's diff easier to read.

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:38
@postamar postamar requested a review from a team as a August 20, 2024 15:38
@postamar postamar force-pushed the postamar_gt_08-20-extract-jdbc_refactor_partitionreader_and_partitionscreator_impls branch from 5904a00 to 6fdccb8 Compare August 20, 2024 15:44
@postamar postamar force-pushed the postamar_gt_08-20-extract-jdbc_rename_partitionscreator_and_partitionreader_impls branch from 456b7b0 to e82f2fe Compare August 20, 2024 15:44
@postamar postamar force-pushed the postamar_gt_08-20-extract-jdbc_refactor_partitionreader_and_partitionscreator_impls branch from 6fdccb8 to 1df0476 Compare August 20, 2024 18:48
@postamar postamar force-pushed the postamar_gt_08-20-extract-jdbc_rename_partitionscreator_and_partitionreader_impls branch from e82f2fe to f3f5fb0 Compare August 20, 2024 18:48
@postamar postamar force-pushed the postamar_gt_08-20-extract-jdbc_refactor_partitionreader_and_partitionscreator_impls branch from 1df0476 to f0fbc36 Compare August 20, 2024 20:56
@postamar postamar force-pushed the postamar_gt_08-20-extract-jdbc_rename_partitionscreator_and_partitionreader_impls branch from f3f5fb0 to 4152603 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