Skip to content

Conversation

@Abacn
Copy link
Contributor

@Abacn Abacn commented Feb 20, 2025

This fixes classic template running on Java 21, which causes same error as apache/beam#33471

@codecov
Copy link

codecov bot commented Feb 20, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 47.18%. Comparing base (ae6134f) to head (0257862).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #2211   +/-   ##
=========================================
  Coverage     47.17%   47.18%           
- Complexity     4053     4055    +2     
=========================================
  Files           876      876           
  Lines         52325    52325           
  Branches       5522     5522           
=========================================
+ Hits          24686    24691    +5     
+ Misses        25882    25878    -4     
+ Partials       1757     1756    -1     
Components Coverage Δ
spanner-templates 69.17% <ø> (+0.01%) ⬆️
spanner-import-export 65.75% <ø> (+0.03%) ⬆️
spanner-live-forward-migration 77.30% <ø> (ø)
spanner-live-reverse-replication 79.47% <ø> (ø)
spanner-bulk-migration 88.32% <ø> (ø)

see 2 files with indirect coverage changes

@Abacn
Copy link
Contributor Author

Abacn commented Feb 20, 2025

R: @shunping

Copy link
Contributor

@shunping shunping left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Abacn Abacn merged commit 797c173 into GoogleCloudPlatform:main Feb 20, 2025
14 of 16 checks passed
@Abacn Abacn deleted the multireleasejar branch February 20, 2025 23:11
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