9426248077
84%

Ran 08 Jun 2024 05:31AM UTC

Jobs 1

Files 55

Run time 1min

Badge

Embed ▾

Committed 07 Jun 2024 11:47PM UTC coverage: 85.257% (+0.07%) from 85.189%

Build # 9426248077

Build Type

push

github

Committed by

web-flow

Commit Message

Improve leadership change signaling (#5504)

Because leadership status can sometimes change very quickly and
raft.updateLeadChange() can 'drop' some lead change signals, especially
with the leadc channel having a capacity of one, which means that
sometimes stream.setLeader(isLeader bool) gets called twice in a row
with the same isLeader value.

This could lead to in some cases the sourcing to 'stall' (not all
messages gets sourced) or 'duplicate messages' (more messages than
expected get sourced).

Changes are increasing the leadc channel's size to 32.

The effects of the change can be demonstrated by running the
TestStreamSourcingScalingSourcingManyBenchmark load test a large number
of times (e.g. 100 times): without these changes the test flaps and
fails between 10 and 20% of the time (most of the time 'stalling' but
also sometimes with 'too many'). With the changes over around a hundred
runs not once did the test fail with 'stalling' and only once did it
fail with 'too many' messages (so there's still a bug/corner case that
could result in 'more than expected' messages to be sourced, although it
seems to be a rare occurence (i.e. 1% of the time)).

This is a replacement for #5496

---------

Signed-off-by: Jean-Noël Moyne <jnmoyne@gmail.com>

Run Details

65670 of 77026 relevant lines covered (85.26%)

946277.52 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	9426248077.1	08 Jun 2024 05:31AM UTC	0	85.26	GitHub Action Run

Source Files on build 9426248077

Detailed source file information is not available for this build.

nats-io / nats-server / 9426248077
84%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 9426248077

nats-io / nats-server / 9426248077 84%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 9426248077

nats-io / nats-server / 9426248077
84%

README BADGES
x