Skip to content

add KMS key support to BigQueryConverters#2024

Merged
Polber merged 2 commits intoGoogleCloudPlatform:mainfrom
Polber:jkinard/bigquery-kms
Nov 27, 2024
Merged

add KMS key support to BigQueryConverters#2024
Polber merged 2 commits intoGoogleCloudPlatform:mainfrom
Polber:jkinard/bigquery-kms

Conversation

@Polber
Copy link
Copy Markdown
Contributor

@Polber Polber commented Nov 19, 2024

This PR adds support for encrypting temporary table created during BigQueryIO read for any templates that use the BigQueryConverters class (v2) to read from BigQuery.

This only applies when using query to read from the BigQuery source

Signed-off-by: Jeffrey Kinard <jeff@thekinards.com>
@Polber Polber self-assigned this Nov 19, 2024
@Polber Polber requested a review from liferoad November 19, 2024 16:45
@Polber Polber added the improvement Making existing code better label Nov 19, 2024
@codecov
Copy link
Copy Markdown

codecov bot commented Nov 19, 2024

Codecov Report

Attention: Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

Project coverage is 45.41%. Comparing base (ff49726) to head (48dbf87).
Report is 9 commits behind head on main.

Files with missing lines Patch % Lines
...oud/teleport/v2/transforms/BigQueryConverters.java 0.00% 3 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2024      +/-   ##
============================================
- Coverage     45.41%   45.41%   -0.01%     
- Complexity     3677     4001     +324     
============================================
  Files           842      842              
  Lines         49970    49970              
  Branches       5261     5262       +1     
============================================
- Hits          22696    22695       -1     
  Misses        25605    25605              
- Partials       1669     1670       +1     
Components Coverage Δ
spanner-templates 66.71% <ø> (-0.01%) ⬇️
spanner-import-export 64.18% <ø> (-0.03%) ⬇️
spanner-live-forward-migration 75.88% <ø> (ø)
spanner-live-reverse-replication 76.65% <ø> (ø)
spanner-bulk-migration 86.37% <ø> (ø)
Files with missing lines Coverage Δ
...oud/teleport/v2/transforms/BigQueryConverters.java 50.21% <0.00%> (ø)

... and 1 file with indirect coverage changes

Signed-off-by: Jeffrey Kinard <jeff@thekinards.com>
@Polber Polber merged commit add3dd7 into GoogleCloudPlatform:main Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Making existing code better size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants