nats-io / gnatsd / 3219

90%

master: 91%

push

travis-ci

Conditional send of routed subs from a go routine

When a route is established, it is possible that each server sends
its list of subscriptions to each other at the same time. Doing
it in place from the readLoop could then cause problems because
each side could reach a point where the outbound socket buffer
is full and no one is dequeuing data (since readLoop is doing
the send of the subs list).
We changed sending this list from a go routine. However, for small
number of subscriptions, it is not required and was causing some
of the tests to fail because of timing issues.

We will now send in place if the estimated size of all protocols
is below a give threshold (1MB).

Signed-off-by: Ivan Kozlovic <ivan@synadia.com>

60 of 60 new or added lines in 1 file covered. (100.0%)

14252 of 15789 relevant lines covered (90.27%)

87611.23 hits per line