Skip to content

Conversation

@dhercher
Copy link
Contributor

support inf and -inf datetime values

@dhercher dhercher requested review from a team as code owners February 18, 2025 21:54
@dhercher dhercher added bug Something isn't working bug-fix labels Feb 18, 2025
@codecov
Copy link

codecov bot commented Feb 18, 2025

Codecov Report

Attention: Patch coverage is 15.38462% with 11 lines in your changes missing coverage. Please review.

Project coverage is 47.05%. Comparing base (7b7081b) to head (b7c5bc6).
Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
...tream/transforms/FormatDatastreamRecordToJson.java 15.38% 11 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2203      +/-   ##
============================================
- Coverage     47.06%   47.05%   -0.02%     
+ Complexity     4376     4373       -3     
============================================
  Files           876      876              
  Lines         52218    52230      +12     
  Branches       5505     5507       +2     
============================================
- Hits          24578    24575       -3     
- Misses        25881    25894      +13     
- Partials       1759     1761       +2     
Components Coverage Δ
spanner-templates 68.90% <ø> (-0.04%) ⬇️
spanner-import-export 65.66% <ø> (-0.08%) ⬇️
spanner-live-forward-migration 76.48% <ø> (-0.06%) ⬇️
spanner-live-reverse-replication 78.80% <ø> (ø)
spanner-bulk-migration 87.94% <ø> (ø)
Files with missing lines Coverage Δ
...tream/transforms/FormatDatastreamRecordToJson.java 56.65% <15.38%> (-0.63%) ⬇️

... and 9 files with indirect coverage changes

@dhercher dhercher merged commit b51c933 into main Feb 19, 2025
32 checks passed
@dhercher dhercher deleted the inf branch February 19, 2025 04:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working bug-fix size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants