Skip to content

test(Spanner): Live: Sqlserver migration integration tests for datatype - #4244

Draft
pratickchokhani wants to merge 2 commits into
GoogleCloudPlatform:mainfrom
pratickchokhani:sqlserver-live-migration-integration-datatype
Draft

pratickchokhani wants to merge 2 commits into
GoogleCloudPlatform:mainfrom
pratickchokhani:sqlserver-live-migration-integration-datatype

Conversation

@pratickchokhani

Copy link
Copy Markdown
Contributor

No description provided.

…g connector implementation, dialect adapter, and configuration mappings
… Server integration tests and data types integration test.
@pratickchokhani
pratickchokhani force-pushed the sqlserver-live-migration-integration-datatype branch from af6c1ab to 3e286ff Compare September 10, 2026 14:44
@pratickchokhani pratickchokhani added the addition New feature or request label Sep 10, 2026
@codecov

codecov Bot commented Sep 10, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 1.17647% with 84 lines in your changes missing coverage. Please review.
✅ Project coverage is 56.21%. Comparing base (d4e3999) to head (3e286ff).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...it/gcp/cloudsql/CloudSqlServerResourceManager.java 0.00% 50 Missing ⚠️
...apache/beam/it/gcp/datastream/SqlServerSource.java 0.00% 23 Missing ⚠️
...m/it/gcp/datastream/DatastreamResourceManager.java 0.00% 11 Missing ⚠️

❌ Your patch check has failed because the patch coverage (1.17%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #4244      +/-   ##
============================================
- Coverage     56.22%   56.21%   -0.02%     
- Complexity     7228     7679     +451     
============================================
  Files          1142     1147       +5     
  Lines         71089    71263     +174     
  Branches       8158     8172      +14     
============================================
+ Hits          39968    40057      +89     
- Misses        28535    28621      +86     
+ Partials       2586     2585       -1     
Components Coverage Δ
spanner-templates 84.46% <100.00%> (+0.08%) ⬆️
spanner-import-export 69.02% <ø> (ø)
spanner-live-forward-migration 88.91% <100.00%> (+0.18%) ⬆️
spanner-live-reverse-replication 80.82% <ø> (ø)
spanner-bulk-migration 88.96% <ø> (ø)
gcs-spanner-dv 88.08% <ø> (ø)
Files with missing lines Coverage Δ
.../org/apache/beam/it/gcp/datastream/JDBCSource.java 19.23% <100.00%> (+3.23%) ⬆️
...m/it/gcp/datastream/DatastreamResourceManager.java 75.39% <0.00%> (-2.72%) ⬇️
...apache/beam/it/gcp/datastream/SqlServerSource.java 0.00% <0.00%> (ø)
...it/gcp/cloudsql/CloudSqlServerResourceManager.java 0.00% <0.00%> (ø)

... and 2 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 changed the title feat(Spanner): Live: Sqlserver migration integration tests for datatype test(Spanner): Live: Sqlserver migration integration tests for datatype Sep 11, 2026
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