12229104575
83%

Ran 09 Dec 2024 05:44AM UTC

Jobs 1

Files 57

Run time 1min

Badge

Embed ▾

Committed 06 Dec 2024 07:35PM UTC coverage: 85.395% (+0.03%) from 85.369%

Build # 12229104575

Build Type

push

github

Committed by

web-flow

Commit Message

[FIXED] Subject state consistency (#6226)

Subject state would not always remain consistent given a specific
pattern of message removals.

There were three bugs:
- `recalculateFirstForSubj` in memstore would do `startSeq+1`, but
filestore would always just start at `mb.first.seq`. These are now
consistent.
- `recalculateFirstForSubj` was not called when `ss.Msgs == 1`, which
could mean we had a stale `ss.FirstSeq` if it needed to be recalculated.
- If after recalculation it turns out `ss.FirstSeq` equals the message
we're trying to remove, we need to `recalculateFirstForSubj` again,
since `ss.Last` is also lazy and could be incorrect.

Apart from that, filestore and memstore are now both equivalent when it
comes to first updating per-subject state and then removing the message,
as well as `removeSeqPerSubject` and how it updates the subject state.


Signed-off-by: Maurice van Veen <github@mauricevanveen.com>

Run Details

68263 of 79938 relevant lines covered (85.39%)

1876173.77 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	12229104575.1	09 Dec 2024 05:44AM UTC	0	85.39	GitHub Action Run

Source Files on build 12229104575

Detailed source file information is not available for this build.

nats-io / nats-server / 12229104575
83%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 12229104575

nats-io / nats-server / 12229104575 83%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 12229104575

nats-io / nats-server / 12229104575
83%

README BADGES
x