1
70%
master: 70%

Ran 10 May 2018 09:13PM UTC

Files 21

Run time 1s

Badge

Embed ▾

Committed 10 May 2018 08:39PM UTC coverage: 72.381% (-0.2%) from 72.552%

Job # 137.1

Build Type

push

travis-ci

Committed by

clumens

Commit Message

Update the compose server to handle multiple simultaneous composes.

Much of the change in this patch is just removing the use of a TVar to
handle synchronization of the worklist between threads, replacing it
with a TMVar.  The TMVar supports retry operations critical for making
this work.

The other big change is replacing composesThread.  That code had a
single thread sitting and watching the worklist, waiting for something
to show up.  That organization made it difficult to have many composes
running at the same time.

Instead, I fork off a number of worker threads immediately.  They all
sit around watching the worklist, waiting for a compose request to
appear.  When a request does appear, a random worker thread will grab it
first and do the compose.  This means I don't ever need to worry about
the size of the in progress map, because the max number of composes
running at a time is constrained by how many workers are spawned.  It
also means the workers are less complicated.

For now, I am leaving max composes at 1.  I have tested two running at a
time, but only on the management side.  I don't know if what's going
into the composes is thread safe.  We need to check that before raising
this number or making it tunable by end users.

Run Details

1575 of 2176 relevant lines covered (72.38%)

1.37 hits per line

weldr / bdcs-api / 137 / 1
70%
master: 70%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 137.1

weldr / bdcs-api / 137 / 1 70% master: 70%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 137.1

weldr / bdcs-api / 137 / 1
70%
master: 70%

README BADGES
x