190
89%
master: 87%

Ran 02 Jan 2018 11:49PM UTC

Jobs 4

Files 28

Run time 12min

Badge

Embed ▾

pending completion

Build # 190

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%)

10534.25 hits per line

Jobs

ID	Job ID	Ran	Coverage
1	190.1	02 Jan 2018 11:49PM UTC	89.28	Travis Job 190.1
2	190.2	02 Jan 2018 11:51PM UTC	89.28	Travis Job 190.2
3	190.3	03 Jan 2018 12:00AM UTC	89.28	Travis Job 190.3
4	190.4	03 Jan 2018 12:02AM UTC	89.28	Travis Job 190.4

Source Files on build 190

Detailed source file information is not available for this build.

codedge-llc / kadabra / 190
89%
master: 87%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 190

codedge-llc / kadabra / 190 89% master: 87%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 190

codedge-llc / kadabra / 190
89%
master: 87%

README BADGES
x