Skip to content

perf: optimize DynamoDB client timeouts and retry config#408

Merged
DJAndries merged 2 commits intomasterfrom
mschfh-sdk-opts
Apr 29, 2026
Merged

perf: optimize DynamoDB client timeouts and retry config#408
DJAndries merged 2 commits intomasterfrom
mschfh-sdk-opts

Conversation

@mschfh
Copy link
Copy Markdown
Contributor

@mschfh mschfh commented Apr 9, 2026

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Optimizes DynamoDB client behavior by tuning HTTP timeouts per request phase and customizing the AWS SDK retry configuration, aiming to reduce hangs and improve performance/reliability for in-region calls.

Changes:

  • Introduces a dedicated newDynamoHTTPClient() with phase-specific transport timeouts and updated connection pooling settings.
  • Adds optional SDK retry logging via LOG_SDK_RETRIES.
  • Configures a custom DynamoDB SDK retryer (attempts/backoff/rate limiting).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread datastore/dynamo.go
Comment thread datastore/dynamo.go
@mschfh mschfh force-pushed the mschfh-sdk-opts branch 2 times, most recently from 6838cb3 to 75d78ef Compare April 10, 2026 13:13
@mschfh mschfh marked this pull request as ready for review April 10, 2026 13:48
@mschfh mschfh requested a review from Copilot April 10, 2026 13:48
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread datastore/dynamo.go Outdated
Comment thread datastore/dynamo.go
Comment thread datastore/dynamo.go
Comment thread datastore/dynamo.go
@DJAndries DJAndries merged commit 8e51cc1 into master Apr 29, 2026
9 of 10 checks passed
@DJAndries DJAndries deleted the mschfh-sdk-opts branch April 29, 2026 19:56
DJAndries added a commit that referenced this pull request Apr 29, 2026
* chore(deps): update dependency go to 1.26 (#402)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update public.ecr.aws/docker/library/golang docker tag to v1.26 (#403)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update actions/setup-go action to v6.4.0 (#400)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix(deps): update module github.com/go-chi/chi/v5 to v5.2.5 (#399)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update actions/checkout action to v6.0.2 (#411)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update golangci/golangci-lint-action action to v9.2.0 (#413)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix(deps): update aws-sdk-go-v2 monorepo (#404)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Add warning logs for items that exceed size limit (#414)

* Add warning logs for items that exceed size limit

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* chore(deps): update alpine docker tag to v3.23 (#401)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* perf: optimize DynamoDB client timeouts and retry config (#408)

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: mschfh <37435502+mschfh@users.noreply.github.com>
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.

3 participants