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.

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

Latency monitoring improvements #347

Open
MRiganSUSX opened this issue Oct 10, 2024 · 0 comments
Open

Latency monitoring improvements #347

MRiganSUSX opened this issue Oct 10, 2024 · 0 comments

Comments

@MRiganSUSX
Copy link
Contributor

New latency monitoring for trigger is introduced with #342.

Some issues:

  • currently updating latency for each TX object, which for TPs is more often than reasonably useful and adds an overhead
  • in cases where the rates are high, we are effectively comparing different objects when we publish our latency_in and latency_out. The latency out will be the latest (TX+1) object that was made while the latency in is the newest incoming TX object. We report the values at the same time, but they do not refer to the same object anymore (different base of comparison). While this is still useful latency reporting, it can produce 'weird' cases for the online monitoring.

possible improvements:

  • using condition variables for updating
  • adding some kind of prescaling / a rate check for the updates
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant