Skip to content

Remove warning message#2173

Merged
dhercher merged 1 commit intomainfrom
dhercher-patch-1
Feb 12, 2025
Merged

Remove warning message#2173
dhercher merged 1 commit intomainfrom
dhercher-patch-1

Conversation

@dhercher
Copy link
Copy Markdown
Contributor

There is no clear value add for the error message, simply pushing the raw data is the best way to handle such custom fields

There is no clear value add for the error message, simply pushing the raw data is the best way to handle such custom fields
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 11, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 46.95%. Comparing base (ccc6edf) to head (7b45ffe).
Report is 3 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2173      +/-   ##
============================================
- Coverage     46.95%   46.95%   -0.01%     
- Complexity     4044     4368     +324     
============================================
  Files           876      876              
  Lines         52190    52189       -1     
  Branches       5501     5501              
============================================
- Hits          24506    24505       -1     
+ Misses        25926    25925       -1     
- Partials       1758     1759       +1     
Components Coverage Δ
spanner-templates 68.88% <ø> (-0.01%) ⬇️
spanner-import-export 65.72% <ø> (-0.02%) ⬇️
spanner-live-forward-migration 76.50% <ø> (ø)
spanner-live-reverse-replication 78.67% <ø> (ø)
spanner-bulk-migration 87.94% <ø> (ø)
Files with missing lines Coverage Δ
...tream/transforms/FormatDatastreamRecordToJson.java 42.71% <ø> (+0.13%) ⬆️

... and 2 files with indirect coverage changes

@dhercher dhercher merged commit b93cad8 into main Feb 12, 2025
32 of 35 checks passed
@dhercher dhercher deleted the dhercher-patch-1 branch February 12, 2025 07:48
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.

3 participants