Page MenuHomePhabricator

[identity] increase connect timeout to 60 seconds
ClosedPublic

Authored by varun on Jul 18 2024, 1:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Sep 6, 6:21 AM
Unknown Object (File)
Mon, Sep 2, 2:54 PM
Unknown Object (File)
Wed, Aug 28, 4:33 PM
Unknown Object (File)
Mon, Aug 26, 12:35 PM
Unknown Object (File)
Mon, Aug 26, 12:35 PM
Unknown Object (File)
Mon, Aug 26, 12:35 PM
Unknown Object (File)
Mon, Aug 26, 12:35 PM
Unknown Object (File)
Fri, Aug 23, 4:41 PM
Subscribers

Details

Summary

the DynamoDB client is timing out if it can't initiate a socket connection in 3.1 seconds. this diff bumps the timeout to 60 seconds.

it's still unclear to me why the db client is experiencing a connect timeout after making hundreds of API calls. i would assume that it's just reusing connections from the underlying connection pool...

but this change resolved the error i was seeing and identity was able to respond to my getDeviceListsForUsers request successfully

Test Plan

applied change to staging, confirmed that i got a successful response when requesting device lists for 12,000 user IDs. request took 33 seconds to complete

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable