Skip to content

Conversation

@k1LoW
Copy link
Owner

@k1LoW k1LoW commented Sep 12, 2025

This pull request migrates the DynamoDB integration from the deprecated AWS SDK v1 to AWS SDK v2 across the codebase. The update includes changes to imports, client initialization, method calls, and test setup, as well as updating related dependencies in go.mod. The migration ensures compatibility with the latest AWS SDK and improves maintainability.

AWS SDK v2 Migration:

  • Updated all imports and type usages from github.com/aws/aws-sdk-go (v1) to github.com/aws/aws-sdk-go-v2 (v2) in datasource/aws.go, drivers/dynamo/dynamo.go, and drivers/dynamo/dynamo_test.go. This includes switching to new client types and method signatures. [1] [2] [3]
  • Refactored DynamoDB client initialization and method calls to use the v2 SDK patterns, such as dynamodb.NewFromConfig, ListTables, and DescribeTable, and updated how endpoints and regions are configured. [1] [2] [3] [4]
  • Adjusted helper functions to use v2 SDK types, such as types.TableDescription and related field access. [1] [2]

Dependency Updates:

  • Bumped AWS SDK v2 and related dependencies to newer versions in go.mod, ensuring compatibility with the updated code.

@k1LoW k1LoW self-assigned this Sep 12, 2025
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions
Copy link
Contributor

Code Metrics Report

main (ef9332c) #743 (1339555) +/-
Coverage 54.2% 52.7% -1.6%
Code to Test Ratio 1:0.4 1:0.4 +0.0
Details
  |                    | main (ef9332c) | #743 (1339555) |  +/-  |
  |--------------------|----------------|----------------|-------|
- | Coverage           |          54.2% |          52.7% | -1.6% |
  |   Files            |             54 |             54 |     0 |
  |   Lines            |           5050 |           5063 |   +13 |
- |   Covered          |           2738 |           2669 |   -69 |
+ | Code to Test Ratio |          1:0.4 |          1:0.4 |  +0.0 |
  |   Code             |          10884 |          10902 |   +18 |
+ |   Test             |           4699 |           4707 |    +8 |

Code coverage of files in pull request scope (43.9% → 31.0%)

Files Coverage +/- Status
datasource/aws.go 0.0% 0.0% modified
datasource/datasource.go 49.1% -0.6% affected
datasource/gcp.go 10.6% -27.7% affected
drivers/bq/bq.go 0.0% -91.3% affected
drivers/databricks/databricks.go 15.0% 0.0% modified
drivers/dynamo/dynamo.go 88.5% -1.1% modified

Reported by octocov

@k1LoW k1LoW merged commit 86a5b18 into main Sep 12, 2025
4 checks passed
@k1LoW k1LoW deleted the fix-lint-warn branch September 12, 2025 14:14
@github-actions github-actions bot mentioned this pull request Sep 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants