2708
86%
dev: 88%

Ran 26 Oct 2016 08:51PM UTC

Jobs 4

Files 42

Run time 2min

Badge

Embed ▾

pending completion

Build # 2708

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

4120 of 4752 relevant lines covered (86.7%)

10853.24 hits per line

Jobs

ID	Job ID	Ran	Coverage
1	2708.1	26 Oct 2016 08:52PM UTC	86.38	Travis Job 2708.1
2	2708.2	26 Oct 2016 08:53PM UTC	86.28	Travis Job 2708.2
3	2708.3	26 Oct 2016 08:51PM UTC	86.13	Travis Job 2708.3
4	2708.4	26 Oct 2016 08:54PM UTC	86.34	Travis Job 2708.4

Source Files on build 2708

Detailed source file information is not available for this build.

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

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 2708

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

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 2708

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

README BADGES
x