2
87%
master: 87%

Ran 04 Dec 2017 10:12PM UTC

Files 28

Run time 1s

Badge

Embed ▾

Committed 04 Dec 2017 10:10PM UTC coverage: 89.276% (+0.7%) from 88.556%

Job # 167.2

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

fix: memory leak on connection crashes (#23)

* feat: better supervision structure

* chore: bumped kadabra to v0.3.5

Also: better test coverage

* feat: entirely new supervision strategy

Kadabra.Supervisor starts a ConnectionSupervisor and StreamSupervisor.
ConnSupervisor manages encoder/decoder tables and the Connection worker
with a :one_for_all restart strategy. StreamSupervisor manages the
streams with :one_for_one restart.

To keep the API the same, Kadabra.Supervisor.start_link returns the pid
of the supervisor itself. The Connection worker is keyed in the registry
by this supervisor pid. The user still makes calls with the returned
pid, but it's forwarded with a via_tuple to the proper worker. Unsure
if this might impact performance.

Possible TODO: Consider keying everything by supervisor pid instead of
the generated ref. While conflicts should be virtually nonexistent, this
would guarantee it.

* chore: update CHANGELOG

* fix: removed unused ConnectionSupervisor.start_connection

Run Details

333 of 373 relevant lines covered (89.28%)

2626.48 hits per line

codedge-llc / kadabra / 167 / 2
87%
master: 87%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 167.2

codedge-llc / kadabra / 167 / 2 87% master: 87%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 167.2

codedge-llc / kadabra / 167 / 2
87%
master: 87%

README BADGES
x