fix(assetlibrary): add retry mechanism for connection error#178
fix(assetlibrary): add retry mechanism for connection error#178
Conversation
|
Thanks for the PR @yuma124 ! I'll aim to review it by end-of-week |
|
I feel like a more proper fix here would be to open the DB connection outside of the lambda handler (from here):
|
+1 to this. |
|
@aaronatbissell @ts-amz Is it necessary to open the DB connection outside of the Lambda handler? If we refer to the Neptune sample code here, we can see that they also open the DB connection in the lambda handler and reuse with each handler call. If there are any specific concerns or aspects that you think I might have overlooked, please let me know. |
Description
Since #168, we have occasionally encountered database connection errors. These errors are related to database connectivity, and in my opinion they should be retried in the AssetLibrary. In this Pull Request, I've added retry mechanism for connection errors.
Type of change
Submission Checklist
Additional Notes: