Skip to content

Bugfix/draining paged queues - #13

Merged
martikan merged 4 commits into
mainfrom
bugfix/draining-paged-queues
Jul 17, 2026
Merged

Bugfix/draining paged queues#13
martikan merged 4 commits into
mainfrom
bugfix/draining-paged-queues

Conversation

@martikan

Copy link
Copy Markdown
Owner
  • fix draining of paged queues
  • add drift detection for queues

martikan added 2 commits July 15, 2026 20:12
It could happen, that the broker paged the messages, and during that
time when it started to read the paged messages.. the draining was
stopped, because it thought that the queue is already empty.
the draining process had to be hardened, so the whole queue will be
drained doesn't matter how big is it and paged or not.
@martikan martikan self-assigned this Jul 16, 2026
@martikan martikan added the bug Something isn't working label Jul 16, 2026
@martikan martikan linked an issue Jul 16, 2026 that may be closed by this pull request
@martikan martikan added the feature New feature or request label Jul 16, 2026
@github-actions

Copy link
Copy Markdown

📋 Coverage Report

Total Coverage: 86.6%

Worst Covered Files

Status File Coverage
🟡 github.com/martikan/artemisctl/internal/journal/salvage.go 71.69%
🟡 github.com/martikan/artemisctl/internal/broker/browse.go 72.37%
🟡 github.com/martikan/artemisctl/internal/broker/coreconvert.go 72.73%
🟡 github.com/martikan/artemisctl/internal/broker/client.go 76.47%
🟡 github.com/martikan/artemisctl/internal/cli/redeliver.go 76.47%
🟡 github.com/martikan/artemisctl/internal/brokertest/broker.go 76.92%

@martikan
martikan merged commit 5b3a438 into main Jul 17, 2026
2 checks passed
@martikan
martikan deleted the bugfix/draining-paged-queues branch July 17, 2026 06:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Issue with draining paged queues and add drift detection also

1 participant