541
73%

Ran 25 Sep 2019 07:54AM UTC

Jobs 5

Files 116

Run time 6min

Badge

Embed ▾

Committed 15 Sep 2019 03:41PM UTC coverage: 69.408% (+3.9%) from 65.471%

Build # 541

Build Type

cron

travis-ci

Committed by

jcchavezs

Commit Message

Fix HTTP reporter potential unbounded goroutine creation (#146)

* Fix HTTP reporter potential unbounded goroutine creation

Currently the HTTP reporter spawns a new goroutine each time a new batch needs
to be sent to the server. Execution of those goroutines gets serialized through
the `sendMtx`.

This behavior is problematic when the zipkin server is down and leads to
creation of unbounded number of goroutines each of which will wait for its turn
on the `sendMtx` and then fail.

Fix this by creating one send goroutine to handle sending data to the server.
This also removes the need for the `sendMtx` since `sendBatch` will only be
called by the send goroutine.

Signed-off-by: Slavomir Kaslev <kaslevs@vmware.com>

* Add more HTTP reporter test cases

Add HTTP reporter test cases for BatchInterval() and BatchSize() options
behavior.

Signed-off-by: Slavomir Kaslev <kaslevs@vmware.com>

Run Details

4456 of 6420 relevant lines covered (69.41%)

25924.04 hits per line

Jobs

ID	Job ID	Ran	Coverage
1	541.1	25 Sep 2019 07:57AM UTC	75.35	Travis Job 541.1
2	541.2	25 Sep 2019 07:54AM UTC	81.46	Travis Job 541.2
3	541.3 (GO111MODULE=on)	25 Sep 2019 07:54AM UTC	62.73	Travis Job 541.3
4	541.4 (GO111MODULE=on)	25 Sep 2019 07:55AM UTC	60.65	Travis Job 541.4
5	541.5 (GO111MODULE=on)	25 Sep 2019 08:00AM UTC	60.65	Travis Job 541.5

Source Files on build 541

Detailed source file information is not available for this build.

openzipkin / zipkin-go / 541
73%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 541

openzipkin / zipkin-go / 541 73%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 541

openzipkin / zipkin-go / 541
73%

README BADGES
x