diff CHANGES.txt @ 6565:2c2dbfc332ba

Try to handle multiple connections better. The session database is a hot spot. When multiple requests (e.g. 20) come in at the same time session database contention can get great. The original code didn't retry session database access when the open failed. This resulted in errors at the client. The second pass delayed 0.01 seconds and retried. It was better but we still had multiple second stalls. I think the first request got in, everybody else backed up and then retried at the same time. Again they stepped on each other. With logging I would see many counters go all the way to low single digits or to -1 indicating falure. This pass uses randomint to generate delays from 0-.125 seconds in 5ms increments. This performs better in testing. I rarely saw a counter less than 13 (2 failed retries). Current logging starts after 6 failures and counts down until success or failure.
author John Rouillard <rouilj@ieee.org>
date Thu, 16 Dec 2021 20:02:00 -0500
parents 21c7c2041a4b
children cd408eb748dd
line wrap: on
line diff
--- a/CHANGES.txt	Wed Dec 15 23:52:25 2021 -0500
+++ b/CHANGES.txt	Thu Dec 16 20:02:00 2021 -0500
@@ -59,6 +59,10 @@
 - handle configparser.InterpolationSyntaxError raised if value
   has a single %. Seems to afect python 3 only. Reported by
   nomicon on IRC. (John Rouillard)
+- add random delay to session database retry code between 0 and .125
+  seconds. This seems to help reduce stalled connections when a
+  number of connections are made at the same time. Log remaining
+  retries once 5 of them have been used. (John Rouillard)
 
 Features:
 

Roundup Issue Tracker: http://roundup-tracker.org/