1
72%
master: 72%

Ran 05 Mar 2019 03:40PM UTC

Files 494

Run time 3min

Badge

Embed ▾

Committed 05 Mar 2019 03:40PM UTC coverage: 72.404% (-0.04%) from 72.448%

Job # 379.1

Build Type

push

jenkins

Committed by GitHub

Commit Message

[DLOG] Avoid double read in readahead

There was a bug in the ReadAheadReader whereby, if it was in a paused
state and the last segment it had read had been closed, then when a
writer started writing new segments, the ReadAheadReader would issue a
double read. This would end up triggering an error when it got to the
end of the segment as the issued one of the issued reads would then be
cancelled when the segment was closed.

This fix adds an explicit state machine to the reader and moves all
state transitions to run on the ordered executor, so only one read can
ever be issued at a time.


Reviewers: Enrico Olivelli <eolivelli@gmail.com>, Sijie Guo <sijie@apache.org>

This closes #1973 from ivankelly/dlog-reader-crash-master

Run Details

26497 of 36596 relevant lines covered (72.4%)

0.72 hits per line

apache / bookkeeper / 379 / 1
72%
master: 72%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 379.1

apache / bookkeeper / 379 / 1 72% master: 72%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 379.1

apache / bookkeeper / 379 / 1
72%
master: 72%

README BADGES
x