600
70%
master: 69%

Ran 08 Feb 2022 03:54AM UTC

Jobs 7

Files 165

Run time 1min

Badge

Embed ▾

pending completion

Build # 600

Build Type

push

travis-ci-com

Committed by Rodrigo Tobar

Commit Message

Start communication thread before janitor process

The main server process and the janitor process communicate through a
pair of Queues. The handling of this communication on the main server
process happens on a separate thread, which we used to start after we
spawned the janitor process. This sequence opened the door for a race
condition where the janitor process would try to read from the
corresponding queue, but the main server wouldn't have put any item on
it yet as the communication thread wouldn't have started yet.

This commit fixes this situation by starting the communication thread
before spawning the janitor process. Since we are not waiting for the
communication thread to be up and running (only starting it) there is
still some chance that the original problem could occur, but with a much
lesser probability.

This addresses the problem originally reported in #78.

Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>

Run Details

13672 of 19626 relevant lines covered (69.66%)

4.85 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	600.1 (DB=sqlite3)	08 Feb 2022 03:54AM UTC	165	69.19	Travis Job 600.1
2	600.2 (DB=mysql)	08 Feb 2022 03:56AM UTC	165	69.07	Travis Job 600.2
3	600.3 (DB=postgresql)	08 Feb 2022 03:55AM UTC	165	68.93	Travis Job 600.3
4	600.4	08 Feb 2022 03:55AM UTC	165	69.25	Travis Job 600.4
5	600.5	08 Feb 2022 03:54AM UTC	165	69.33	Travis Job 600.5
6	600.6	08 Feb 2022 03:54AM UTC	165	69.43	Travis Job 600.6
7	600.7	08 Feb 2022 03:55AM UTC	165	69.37	Travis Job 600.7

ICRAR / ngas / 600
70%
master: 69%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 600

ICRAR / ngas / 600 70% master: 69%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 600

ICRAR / ngas / 600
70%
master: 69%

README BADGES
x