TykTechnologies / tyk / 2679 / 1

Ran 26 Jul 2017 02:23PM UTC

Files 84

Run time 4s

Badge

Embed ▾

Committed 26 Jul 2017 02:20PM UTC coverage: 43.746% (-0.005%) from 43.751%

Job # LATEST_GO=true

Build Type

push

travis-ci

Committed by

buger

Commit Message

mw: avoid double goroutine in rate limiting

If UseAsyncSessionWrite is true, DefaultSessionManager.UpdateSession
does the actual store write in a goroutine. Other than that, the
function has very little logic and a json marshal.

As such, it's redundant for this middleware to also use a goroutine. It
would only lead to extra CPU use and delay in writing the session to the
store, given how two goroutines need to be scheduled instead of just
one.

Coverage Stats

5939 of 13576 relevant lines covered (43.75%)

0.47 hits per line

TykTechnologies / tyk / 2679 / 1

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2679.1 (LATEST_GO=true)

TykTechnologies / tyk / 2679 / 1

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2679.1 (LATEST_GO=true)

README BADGES
x