• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

tarantool / tarantool / 9373329469
88%

Build:
DEFAULT BRANCH: master
Ran 04 Jun 2024 08:01PM UTC
Jobs 1
Files 513
Run time 2min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

04 Jun 2024 07:52PM UTC coverage: 87.125% (+0.03%) from 87.098%
9373329469

push

github

Gerold103
relay: do not report vclock[0] anywhere

Remote replica's vclock is given to master to send data starting
from that position. The master does that, but, in order to find
the relevant position in local WAL to start from, the master must
ignore the local rows. Consider them all already "sent". For that
the master replaces the remote vclock[0] with the local vclock[0].
That makes xlog cursor skip all the local rows.

The problem is that this vclock was taken by relay as is, like if
it was truly reported by the replica. It was even saved as the
"last received ACK". Which clearly isn't the case.

When a real ACK was received, it didn't contain anything in
vclock[0], and yet relay "saw" that the previous ACK has
vclock[0] > 0. That looked like the replica went backwards without
even closing connection, which isn't possible. That made the relay
crash from cringe (on assert).

The fix is not to save the local vclock[0] in the last received
ACK.

For GC and xlog cursor the hack is still needed. An option how to
make it easier was to set vclock[0] to INT64_MAX to just never
even bother with any local rows, but that didn't work. Some
assumptions in other places seem to depend on having a proper
local LSN in these places.

Closes #10047

NO_CHANGELOG=the bug wasn't released
NO_DOC=bugfix

67564 of 119999 branches covered (56.3%)

8 of 8 new or added lines in 2 files covered. (100.0%)

102 existing lines in 14 files now uncovered.

100300 of 115122 relevant lines covered (87.12%)

2463391.75 hits per line

Jobs
ID Job ID Ran Files Coverage
1 9373329469.1 04 Jun 2024 08:01PM UTC 0
87.12
GitHub Action Run
Source Files on build 9373329469
Detailed source file information is not available for this build.
  • Back to Repo
  • 1f75231a on github
  • Prev Build on master (#9315966657)
  • Next Build on master (#9396831161)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc