GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — fix: Add connection pre-warming for DynamoDB async client by nithin42 · Pull Request #6711 · feast-dev/feast

Conversation

@nithin42

@nithin42 nithin42 commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

What this PR does / why we need it

The DynamoDB online store async client doesn't establish actual TCP/TLS connections during initialize(), causing the very first get_online_features() call to pay a cold-start overhead (~20ms).
This PR adds a warmup_connections boolean option to DynamoDBOnlineStoreConfig and runs describe_limits() inside a safe try...except block in DynamoDBOnlineStore.initialize() to pre-warm the connections.

Which issue(s) this PR fixes

Fixes #6060

Checks

  • I've made sure the tests are passing.
  • My commits are signed off (git commit -s)
  • My PR title follows conventional commits format

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@nithin42
nithin42 requested a review from a team as a code owner August 7, 2026 19:05
This was referenced Aug 7, 2026
fix: Add DynamoDB connection pre-warming, fix mypy TorchTensor type alias, and fix CI mcp pin mismatch #6710
Closed
Add connection pre-warming for DynamoDB async client #6060
Closed
@nithin42

nithin42 commented Aug 10, 2026

Copy link
Copy Markdown
Contributor Author

@jyejare can you look into fix?
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@codecov-commenter

codecov-commenter commented Aug 11, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 46.82%. Comparing base (98e5bca) to head (5eb6037).
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6711      +/-   ##
==========================================
+ Coverage   46.80%   46.82%   +0.01%     
==========================================
  Files         415      415              
  Lines       50399    50406       +7     
  Branches     7214     7215       +1     
==========================================
+ Hits        23591    23601      +10     
+ Misses      25157    25155       -2     
+ Partials     1651     1650       -1
Flag Coverage Δ
go-feature-server 30.58% <ø> (ø)
python-unit 48.14% <100.00%> (+0.01%) ⬆️
Files with missing lines Coverage Δ
sdk/python/feast/infra/online_stores/dynamodb.py 65.34% <100.00%> (+0.84%) ⬆️

... and 1 file with indirect coverage changes

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 98e5bca...5eb6037. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@ntkathole

ntkathole commented Aug 11, 2026

Copy link
Copy Markdown
Member

@nithin42 you can add this new config in documentation specially at Performance tuning guide - DynamoDB tuning section
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@nithin42

nithin42 commented Aug 11, 2026

Copy link
Copy Markdown
Contributor Author

@ntkathole Done! Documented the new warmup_connections parameter under the DynamoDB tuning sections in both the Performance Tuning Guide and the DynamoDB reference docs. Also added full unit test coverage for the initial connection failure exception path. Thanks!
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@nithin42
nithin42 force-pushed the fix/dynamodb-warmup-connections-only branch from ba7c7bb to cba5604 Compare August 11, 2026 12:35
@nithin42

nithin42 commented Aug 12, 2026

Copy link
Copy Markdown
Contributor Author

@ntkathole @jyejare Hello! I've addressed your feedback by documenting the new warmup_connections parameter in the tuning guides and reference docs, added unit tests covering the exception handling path, and rebased the branch onto master. All CI checks are now passing green. Could you please take a look and review when you have a moment? Thanks!
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@ntkathole ntkathole added the ok-to-test label Aug 14, 2026
ntkathole
ntkathole approved these changes Aug 15, 2026
View reviewed changes
@ntkathole

ntkathole commented Aug 15, 2026

Copy link
Copy Markdown
Member

@nithin42 I am not able to rebase, please rebase and possibly squash commits
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@nithin42
fix: Add connection pre-warming for DynamoDB async client
5eb6037 
Add warmup_connections parameter to DynamoDBOnlineStoreConfig and warm up
connections via describe_limits inside initialize() method to solve cold-start
latency. Also document warmup_connections in the performance tuning guide and
reference docs.

Fixes feast-dev#6060

Signed-off-by: Nithin <[email protected]>
@nithin42
nithin42 force-pushed the fix/dynamodb-warmup-connections-only branch from cba5604 to 5eb6037 Compare August 15, 2026 13:14
@nithin42

nithin42 commented Aug 15, 2026

Copy link
Copy Markdown
Contributor Author

@ntkathole Done! Rebased onto the latest master and squashed into a single clean commit.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@ntkathole
ntkathole merged commit 89240fa into feast-dev:master Aug 15, 2026
25 checks passed

Uh oh!

There was an error while loading. Please reload this page.
@nanohanno nanohanno mentioned this pull request Oct 7, 2026
DynamoDB warmup connections only warms up single connection #6964
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ntkathole ntkathole ntkathole approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

ok-to-test

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add connection pre-warming for DynamoDB async client

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@nithin42 @codecov-commenter @ntkathole
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.