2
66%
master: 73%

Ran 09 Dec 2015 06:05PM UTC

Files 87

Run time 2s

Badge

Embed ▾

Committed 09 Dec 2015 05:26PM UTC coverage: 66.109%. First build

Job # 755.2

Build Type

push

travis-ci

Committed by

MStoykov

Commit Message

reimplement throttle handler

the throttle handler reimplementation has one main strenght - it can use
sendfile. Other benifits is that it removes a dependancy. I would also
argue it's lighter runtime(don't quote me).

In order to use this we use the now new ThrottledWriter which records
how much bytes it has throttled at a given speed and tries to write as
much as it can in order to get to the speed that is requested.

The reason why it's done in the timeoutConn instead of in the handler
given that it requires the whole getting the correct connection for each
request is:
http.ResponseWriter.ReadFrom does os.Stat on each io.LimitedReader it
gets. This adds up pretty quickly to hundred of thousands of os.Stat, which
both slow down the whole application, and make a lot of objects.

Given the fact that io.LimitedReader is the only way to limit the amount
of data the sendfile sends at a time and there is no way to go around
http.ResponseWriter.ReadFrom without Hijacking the connection, the
current solution emerged.

Additional benifit is that other handlers can now throttle as well.

Run Details

2567 of 3883 relevant lines covered (66.11%)

2770.06 hits per line

ironsmile / nedomi / 755 / 2
66%
master: 73%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 755.2

ironsmile / nedomi / 755 / 2 66% master: 73%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 755.2

ironsmile / nedomi / 755 / 2
66%
master: 73%

README BADGES
x