Prioritize Resend, if configured. #688
Conversation
…y app to catch up by scheduling resend responses first over new/pending messages.
chrjohn
changed the title
|
This probably relates to #271 @philipwhiuk you created #271 but this seems to be the other way round. You wanted to throttle resend request responses, this PR right here wants to prioritize them. So it seems we have two competing concepts here. Any thoughts? |
|
In terms of priority that wasn't really a concern it was more about rate-limiting - I imagine the counter-party would have had a similar issue if the actual flow was high. On an app level we try not to route stuff to sessions that are pending a resend. My intention is to spend some of next week on re-implementing some read throttling code we had in our app. I wanted to rebase our changes first. At that point it seemed sensible to upstream as much as possible to minimise the work we have to do to maintain our internal fork (which is necessary in order to fix production issues ASAP). But done properly it should allow a write throttle too. |
Prioritize Resend, if configured, allows lagging counter-party app to catch up by scheduling resend responses first over new/pending messages.