Repository navigation
Conversation
When logging a protocol error, show the data passed as well. Also, provide better logging during sessions management by the processor.
jacomago
reviewed
Oct 7, 2026
Collaborator
|
Can you describe the recciever behaviour in question here... while we are deploying the new recciever here I am interested in seeing if the same issue is present. |
Several messages that describe exceptional circumstances are being hidden by the debug level log severity. These are being upgraded. General principle is that messages that occur once-per-record should be debug, but once-per-ioc is fine, and anything that is exceptional even on a per-record basis can be info or warning.
simon-ess
force-pushed
the
more_logging_sheesh
branch
from
October 8, 2026 10:34
203ad5d to
89a78d1
Compare
|
jacomago
approved these changes
Oct 8, 2026
Contributor
Author
|
@shroffk - What don't have a full picture of what is happening. But broadly what we are seeing is that on our busiest recceiver (2500+ IOCs), the recceiver suddenly starts dropping connections from the IOCs, causing a flood of connect/disconnect logs and ultimately a memory spike causing it to crash. We do not have a clear sense of why this is happening, and are trying to sort it out. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



We are trying to better understand the life cycle of IOC connections, having seen some very odd behaviour for the recceiver under load.