tarantool / tarantool / 9001074088

86%

master: 88%

Build Type

push

github

Committed by sergepetrenko

Commit Message

relay: fix timeout on large xlog subscribe start

When a replica subscribes, it might in the beginning try to
position its reader cursor to the end of a large xlog file.

Positioning inside of this file can take significant time during
which the WAL reader yielded and tried to send heartbeats, but
couldn't, because the relay thread wasn't communicating with the
TX thread.

When there are no messages from TX for too long time, the
heartbeats to the replica are not being sent
(commit 56571d831 ("raft: make
followers notice leader hang")).

The relay must communicate with the TX thread even when subscribe
is just being started and opens a large xlog file.

This isn't the first time when the missing heartbeats result into
timeouts. See more here:

- commit 30ad4a554 ("relay: yield
    explicitly every N sent rows").

- commit 17289440b ("recovery: make
    it yield when positioning in a WAL").

- commit ee6de025f ("relay: send
    heartbeats while reading a WAL").

Given that this is fixed fourth time, it might suggest that the
relay has not the best architecture having some slight drawbacks.
See more in #9968.

Closes #9094

NO_DOC=bugfix

(cherry picked from commit f7e6686a4)

Run Details

62598 of 113808 branches covered (55.0%)

15 of 15 new or added lines in 1 file covered. (100.0%)

37 existing lines in 11 files now uncovered.

93479 of 108933 relevant lines covered (85.81%)

2585394.7 hits per line