167
87%

Ran 04 Dec 2017 10:12PM UTC

Jobs 4

Files 28

Run time 18s

Badge

Embed ▾

pending completion

Build # 167

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

337 of 373 relevant lines covered (90.35%)

10560.12 hits per line

Jobs

ID	Job ID	Ran	Coverage
1	167.1	04 Dec 2017 10:12PM UTC	89.28	Travis Job 167.1
2	167.2	04 Dec 2017 10:12PM UTC	89.28	Travis Job 167.2
3	167.3	04 Dec 2017 10:12PM UTC	89.28	Travis Job 167.3
4	167.4	04 Dec 2017 10:12PM UTC	90.35	Travis Job 167.4

Source Files on build 167

Detailed source file information is not available for this build.

codedge-llc / kadabra / 167
87%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 167

codedge-llc / kadabra / 167 87%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 167

codedge-llc / kadabra / 167
87%

README BADGES
x