10105827586
83%

Ran 26 Jul 2024 05:36AM UTC

Jobs 1

Files 56

Run time 1min

Badge

Embed ▾

Committed 25 Jul 2024 03:26PM UTC coverage: 85.22% (+0.1%) from 85.106%

Build # 10105827586

Build Type

push

github

Committed by

web-flow

Commit Message

Fix `WorkQueuePolicy` retention issue (#5697)

The `isInterestRetention` function, which gates whether or not we run
`checkInterestState`, was matching for both `InterestPolicy` and
`WorkQueuePolicy`.

This meant that a work queue stream could throw away messages for which
there was no consumer interest as the consumer ack floors for other
subjects moved up.

This PR changes `isInterestRetention` to match only `InterestPolicy`
which fixes the issue. Notably this doesn't change the `ackMsg`
behaviour which will continue to remove messages once they have been
acked.

Signed-off-by: Neil Twigg <neil@nats.io>

Run Details

66124 of 77592 relevant lines covered (85.22%)

1220919.26 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	10105827586.1	26 Jul 2024 05:36AM UTC	0	85.22	GitHub Action Run

Source Files on build 10105827586

Detailed source file information is not available for this build.

nats-io / nats-server / 10105827586
83%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 10105827586

nats-io / nats-server / 10105827586 83%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 10105827586

nats-io / nats-server / 10105827586
83%

README BADGES
x