2
86%
dev: 88%

Ran 26 Oct 2016 08:53PM UTC

Files 42

Run time 4s

Badge

Embed ▾

Committed 26 Oct 2016 08:48PM UTC coverage: 86.279%. First build

Job # 2708.2

Build Type

push

travis-ci

Committed by

prashantv

Commit Message

relay: Fail calls if the dest connection's sendCh is full

The current behaviour tries to write the frame to sendCh forever, which
can cause issues if the connection is stalled. The current behaviour
with a stalled connection would be:
- S1 calls S2, S2's connection gets stalled (bug in S2, or TCP issue)
- Relay reads from S1's connection, and in the same goroutine tries to
  write it to S2's sendCh
- If the connection is stalled, the write would not make any progress.
- All calls from S1 on that connection are now "stuck" behind the
  stalled connection. The relay would not even see frames (hence no
  stats would be incremented), and no progress would be made till the
  connection was killed.

Instead of trying to write to sendCh forever, we should fail the call
immediately if sendCh is at capacity. Instead of the server timing out,
they will instead see an error from the relay and we'll have stats on
the relay side to detect this.

Run Details

4081 of 4730 relevant lines covered (86.28%)

2732.21 hits per line

uber / tchannel-go / 2708 / 2
86%
dev: 88%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2708.2

uber / tchannel-go / 2708 / 2 86% dev: 88%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2708.2

uber / tchannel-go / 2708 / 2
86%
dev: 88%

README BADGES
x