-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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.
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
basic prototype to demonstrate dlqing stale messages #82029
base: master
Are you sure you want to change the base?
Conversation
the goal of this change is prioritise new messages during incident recovery, and reroute stale messages for later processing. this PR is a prototype to demonstrate one possible approach using a DLQ
Codecov ReportAttention: Patch coverage is ✅ All tests successful. No failed tests found.
Additional details and impacted files@@ Coverage Diff @@
## master #82029 +/- ##
==========================================
+ Coverage 80.36% 80.42% +0.06%
==========================================
Files 7290 7284 -6
Lines 321739 321195 -544
Branches 21009 20932 -77
==========================================
- Hits 258567 258328 -239
+ Misses 62766 62465 -301
+ Partials 406 402 -4 |
This pull request has gone three weeks without activity. In another week, I will close it. But! If you comment or otherwise update it, I will reset the clock, and if you add the label "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
the goal of this change is prioritise new messages during incident recovery, and reroute stale messages for later processing.
this PR is a prototype to demonstrate one possible approach using a DLQ