Skip to content

Conversation

@VardhanThigle
Copy link
Contributor

No description provided.

@VardhanThigle VardhanThigle changed the title Json rounding [DRAFT UNDER TEST - NOT FOR REVIEW] Logging Failed Mutations to GCS. Nov 19, 2025
@codecov
Copy link

codecov bot commented Nov 19, 2025

Codecov Report

❌ Patch coverage is 80.55556% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 50.43%. Comparing base (a73e0f0) to head (e63afba).
⚠️ Report is 14 commits behind head on main.

Files with missing lines Patch % Lines
...le/cloud/teleport/spanner/TextImportTransform.java 85.71% 4 Missing ⚠️
...gle/cloud/teleport/spanner/TextImportPipeline.java 0.00% 3 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #3004      +/-   ##
============================================
+ Coverage     50.26%   50.43%   +0.17%     
- Complexity     5021     5030       +9     
============================================
  Files           967      969       +2     
  Lines         59268    59572     +304     
  Branches       6458     6495      +37     
============================================
+ Hits          29789    30048     +259     
- Misses        27376    27404      +28     
- Partials       2103     2120      +17     
Components Coverage Δ
spanner-templates 70.66% <80.55%> (+0.21%) ⬆️
spanner-import-export 69.08% <80.55%> (+0.07%) ⬆️
spanner-live-forward-migration 80.04% <ø> (+0.35%) ⬆️
spanner-live-reverse-replication 77.46% <ø> (+0.38%) ⬆️
spanner-bulk-migration 88.28% <ø> (-0.06%) ⬇️
Files with missing lines Coverage Δ
...le/cloud/teleport/spanner/CSVRecordToMutation.java 83.06% <100.00%> (+3.06%) ⬆️
...gle/cloud/teleport/spanner/TextImportPipeline.java 0.00% <0.00%> (ø)
...le/cloud/teleport/spanner/TextImportTransform.java 50.00% <85.71%> (+6.52%) ⬆️

... and 13 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.

@VardhanThigle VardhanThigle force-pushed the jsonRounding branch 3 times, most recently from 01e4dec to 5d7d118 Compare November 24, 2025 06:38
@VardhanThigle VardhanThigle force-pushed the jsonRounding branch 3 times, most recently from 824764f to 041695b Compare November 25, 2025 09:04
@VardhanThigle VardhanThigle force-pushed the jsonRounding branch 10 times, most recently from 78f7da0 to 51b9b34 Compare December 3, 2025 01:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant