Skip to content

test(Spanner): Reverse: implement comprehensive Spanner-to-SQLServer integration test suite and update source connector logic - #4212

Draft
pratickchokhani wants to merge 7 commits into
GoogleCloudPlatform:mainfrom
pratickchokhani:sqlserver-reverse-migration-integration
Draft

test(Spanner): Reverse: implement comprehensive Spanner-to-SQLServer integration test suite and update source connector logic#4212
pratickchokhani wants to merge 7 commits into
GoogleCloudPlatform:mainfrom
pratickchokhani:sqlserver-reverse-migration-integration

Conversation

@pratickchokhani

Copy link
Copy Markdown
Contributor

No description provided.

@pratickchokhani pratickchokhani added the addition New feature or request label Sep 2, 2026
@codecov

codecov Bot commented Sep 2, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 82.42574% with 142 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.42%. Comparing base (9f87f58) to head (9546820).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...plates/source/sqlserver/SQLServerDMLGenerator.java 64.80% 39 Missing and 24 partials ⚠️
...rce/sqlserver/SQLServerSpToSrcSourceConnector.java 39.28% 34 Missing ⚠️
...lectadapter/sqlserver/SqlServerDialectAdapter.java 91.42% 6 Missing and 12 partials ⚠️
...er/migrations/connection/JdbcConnectionHelper.java 44.00% 10 Missing and 4 partials ⚠️
...eport/v2/templates/transforms/AssignShardIdFn.java 0.00% 6 Missing ⚠️
...rowmapper/provider/SqlServerJdbcValueMappings.java 97.16% 1 Missing and 2 partials ⚠️
.../v2/spanner/migrations/avro/AvroToValueMapper.java 97.29% 1 Missing and 1 partial ⚠️
...rce/sqlserver/SqlServerSrcToSpSourceConnector.java 98.86% 0 Missing and 1 partial ⚠️
...r/sourceddl/SQLServerInformationSchemaScanner.java 97.82% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##               main    #4212       +/-   ##
=============================================
+ Coverage     33.92%   63.42%   +29.50%     
- Complexity      505     2933     +2428     
=============================================
  Files           215      570      +355     
  Lines         13148    33577    +20429     
  Branches       1308     3787     +2479     
=============================================
+ Hits           4460    21297    +16837     
- Misses         8321    11188     +2867     
- Partials        367     1092      +725     
Components Coverage Δ
spanner-templates 84.13% <82.42%> (∅)
spanner-import-export ∅ <ø> (∅)
spanner-live-forward-migration 88.67% <88.43%> (∅)
spanner-live-reverse-replication 79.88% <69.15%> (∅)
spanner-bulk-migration 89.26% <93.10%> (∅)
gcs-spanner-dv 88.01% <88.43%> (∅)
Files with missing lines Coverage Δ
...v2/reader/io/jdbc/iowrapper/config/SQLDialect.java 100.00% <100.00%> (ø)
...oud/teleport/v2/source/SourceConnectorFactory.java 100.00% <100.00%> (ø)
...apper/config/defaults/SqlServerConfigDefaults.java 100.00% <100.00%> (ø)
...ort/v2/spanner/migrations/constants/Constants.java 0.00% <ø> (ø)
...r/migrations/source/config/SourceConfigParser.java 79.54% <ø> (ø)
...2/spanner/migrations/source/config/SourceType.java 100.00% <100.00%> (ø)
...ates/dbutils/processor/SourceProcessorFactory.java 96.29% <100.00%> (ø)
...rce/sqlserver/SqlServerSrcToSpSourceConnector.java 98.86% <98.86%> (ø)
...r/sourceddl/SQLServerInformationSchemaScanner.java 97.82% <97.82%> (ø)
.../v2/spanner/migrations/avro/AvroToValueMapper.java 95.34% <97.29%> (ø)
... and 6 more

... and 435 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pratickchokhani
pratickchokhani force-pushed the sqlserver-reverse-migration-integration branch 19 times, most recently from 53c4277 to 5eec168 Compare September 7, 2026 09:11
pratickchokhani and others added 7 commits September 7, 2026 14:42
…g connector implementation, dialect adapter, and configuration mappings
…cryption, improved schema handling, and robust byte buffer conversion
…generation support for Spanner-to-SourceDB migration
…integration test suite and update source connector logic
@pratickchokhani
pratickchokhani force-pushed the sqlserver-reverse-migration-integration branch from 5eec168 to 9546820 Compare September 7, 2026 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

addition New feature or request size/XXL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant