248
69%
master: 78%

Ran 29 Mar 2019 12:42PM UTC

Jobs 1

Files 35

Run time 3s

Badge

Embed ▾

pending completion

Build # 248

Build Type

push

travis-ci

Committed by

nicktorwald

Commit Message

Race condition in TarantoolClientImpl

- Avoid a possible race between reading, writing and reconnecting
threads when a reconnection process is started.
It might have happened that the lagged thread (reading or writing) could
reset the state to RECONNECT after the reconnecting thread has already
started and set the state to 0. As a result, all next attempts to
reconnect will never happen. Now the reconnect thread holds on the state
as long as it is required.

- Avoid another possible race between reading and writing threads when
they are started during the reconnection process.
It might have happened that one of the threads crashed when it was
starting and another slightly lagged thread set up its flag. It could
have led that the reconnecting thread saw RECONNECT + R/W state instead
of pure RECONNECT. Again, this case broke down all next reconnection
attempts. Now reading and writing threads take into account whether
RECONNECT state is already set or not.

- Avoid LockSupport class usage for a thread to be suspended and woken
up. Actually, LockSupport is more like an internal component to build
high-level blocking primitives. It is not recommended using this class
directly. It was replaced by ReentrantLock.Condition primitive based
on LockSupport but which has proper LockSupport usage inside.

Fixes: #142
Affects: #34, #136

Run Details

1440 of 2267 relevant lines covered (63.52%)

0.64 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
3	248.3	29 Mar 2019 12:42PM UTC	35	63.52	Travis Job 248.3

tarantool / tarantool-java / 248
69%
master: 78%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 248

tarantool / tarantool-java / 248 69% master: 78%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 248

tarantool / tarantool-java / 248
69%
master: 78%

README BADGES
x