Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Properly flush unique queues on startup #23154

Merged
merged 12 commits into from
Feb 28, 2023
Merged

Properly flush unique queues on startup #23154

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
a610c89
Properly flush unique queues on startup
zeripath Feb 25, 2023
9e5ea4c
Update modules/queue/unique_queue_disk_channel_test.go
zeripath Feb 25, 2023
7c99ea6
And set the internal queue name for the queues
zeripath Feb 26, 2023
b8fa5cb
Improve test to include some interleaving of queues
zeripath Feb 26, 2023
2854448
Queues need to be named the same - fix TestPersistsableChannelQueue
zeripath Feb 26, 2023
a233641
adjust logging slightly to remove some more spurious logs
zeripath Feb 26, 2023
7396bf7
try to avoid warning and lock issues
zeripath Feb 26, 2023
efe654d
avoid logging the channel early shutdown message if the q is not empty
zeripath Feb 26, 2023
45f417e
Merge branch 'main' into fix-23050-ensure-that-unique-queues-get-prop…
zeripath Feb 26, 2023
8cbcff2
fix testcase
zeripath Feb 26, 2023
877136d
Merge branch 'main' into fix-23050-ensure-that-unique-queues-get-prop…
zeripath Feb 27, 2023
a2705dd
Merge branch 'main' into fix-23050-ensure-that-unique-queues-get-prop…
techknowlogick Feb 28, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'main' into fix-23050-ensure-that-unique-queues-get-prop… 
…erly-restored-on-restart
  • Loading branch information
@zeripath
zeripath authored Feb 26, 2023
commit 45f417e447a5defe137c6b967c168a4082540b4e

This merge commit was added into this branch cleanly.

There are no new changes to show, but you can still view the diff.