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.

Sign up for GitHub

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

Jump to bottom

[Destination MSSQL] add dedup #51612

Draft
wants to merge 34 commits into
base: master
Choose a base branch
from
Draft

[Destination MSSQL] add dedup #51612

wants to merge 34 commits into from
+2,583 −42

Conversation

gosusnp
Copy link
Contributor

@gosusnp gosusnp commented Jan 17, 2025

What

How

Review guide

User Impact

Can this PR be safely reverted and rolled back?

  • YES 💚
  • NO ❌

jdpgrailsdev and others added 25 commits January 17, 2025 11:22
@jdpgrailsdev
feat: implement converters for airbyte type/value to SQL
9449578
@jdpgrailsdev
Revert "feat: implement converters for airbyte type/value to SQL"
a3d26cc 
This reverts commit 90aa35e.
@jdpgrailsdev
feat: implement converters for airbyte type/value to SQL (#49924)
09a9483
@jdpgrailsdev
fix: add missing import (#49945)
81f6e81
@jdpgrailsdev
Remove duplicate class
a7bc094
@gosusnp @jdpgrailsdev
chore: destination MSSQL v2 add spec (#49947)
8de5f77 
Co-authored-by: Jonathan Pearlin <[email protected]>
@jdpgrailsdev
Test data source configuration (#49951)
ff6a8ba
@gosusnp @jdpgrailsdev
chore: add ssl method to dest mssql v2 spec (#49956)
086354f
@gosusnp @jdpgrailsdev
chore: destination mssqlv2 implement configuration and datasource (#4…
e2fe71c 
…9958)
@gosusnp @jdpgrailsdev
chore: mssqlv2 implement check (#49965)
09354e6
@gosusnp @jdpgrailsdev
chore(destination-mssql-v2): add datasource url tests (#50426)
60b31c8 
Co-authored-by: Jonathan Pearlin <[email protected]>
@jdpgrailsdev
refactor: use Kotlin extension function
f183da0
@jdpgrailsdev
feat: use mssql server test container in integration tests (#50869)
a2b6bc9
@gosusnp @jdpgrailsdev
[Destination MSSQLv2] Implement typed insertions (#50434)
45b54f8 
Co-authored-by: Jonathan Pearlin <[email protected]>
@gosusnp @jdpgrailsdev @octavia-squidington-iii
[Destination MSSQL] schema management (#51027)
27a1359 
Co-authored-by: Jonathan Pearlin <[email protected]>
Co-authored-by: Jonathan Pearlin <[email protected]>
Co-authored-by: Octavia Squidington III <[email protected]>
@jdpgrailsdev
fix: use prepared statement instead of dynamic string (#51597)
30568f2
@gosusnp @octavia-squidington-iii @jdpgrailsdev
[Destination MSSQL] Simple truncate (#51528)
962fd06 
Co-authored-by: Octavia Squidington III <[email protected]>
@gosusnp @octavia-squidington-iii @jdpgrailsdev
[Destination MSSQL] fix statement syntax (#51598)
7615ee9 
Co-authored-by: Octavia Squidington III <[email protected]>
@jdpgrailsdev
chore: cleanup/fix warnings
5339a2e
@jdpgrailsdev
fix: use testcontainer for write tests (#51594)
28005f5
@jdpgrailsdev
chore: update dependencies
ac6afbe
@jdpgrailsdev
fix: use correct type in tests
efeb04d
@gosusnp
use table value insert
40b574b
@gosusnp
handle dedup
f833059
@gosusnp
follow cdk changes
613dcab
@gosusnp gosusnp requested a review from jdpgrailsdev January 17, 2025 19:29
@vercel Vercel
Copy link

vercel bot commented Jan 17, 2025 •
edited
Loading

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 Jan 17, 2025 11:19pm

@octavia-squidington-iii octavia-squidington-iii added area/connectors Connector related issues CDK Connector Development Kit connectors/destination/mssql-v2 labels Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jdpgrailsdev jdpgrailsdev Awaiting requested review from jdpgrailsdev

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
area/connectors Connector related issues CDK Connector Development Kit connectors/destination/mssql-v2
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@gosusnp @octavia-squidington-iii @jdpgrailsdev