-
Notifications
You must be signed in to change notification settings - Fork 21.1k
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
Account for race condition in the design pattern recommendations for ingestion delay #122521
base: main
Are you sure you want to change the base?
Conversation
…ing to consider a race condition I describe the issue in detail in https://medium.com/@nwcodex/ingestion-delay-variance-race-conditions-in-seim-systems-8ffc3e298f32
@microsoft-github-policy-service agree |
@NWc0de : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
Learn Build status updates of commit 221d5fc: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
Can you review the proposed changes? When the changes are ready for publication, add a #label:"aq-pr-triaged" |
#label:"backlog-item-created" |
Thanks for the recommendation @NWc0de! We're going to look into this and pull your changes into our own branch to make any required updates, and I've opened a backlog item to handle the work. |
No problem, thank you for the review (: I've removed the references to Splunk (sorry about that 😅 , typo left over from when I wrote this up to send a colleague about the way it may show up in our other SIEMS). |
Learn Build status updates of commit 72b0c4b: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
I describe the issue in detail in https://medium.com/@nwcodex/ingestion-delay-variance-race-conditions-in-seim-systems-8ffc3e298f32. Important aspect here is to let the user know they should consider and design with the race in mind. I don't yet have a good design or conceptual fix.