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

Jump to bottom

Allow monitoring of operational statistics #7

Closed
tkowalcz opened this issue Nov 19, 2020 · 0 comments · Fixed by #20
Closed

Allow monitoring of operational statistics #7

tkowalcz opened this issue Nov 19, 2020 · 0 comments · Fixed by #20
Labels
core Relates to Core component

Comments

@tkowalcz
Copy link
Owner

Expose for monitoring statistics such as number of calls, errors, data size, connections stats, timing info etc.
@tkowalcz tkowalcz added the core Relates to Core component label Nov 19, 2020
tkowalcz added a commit that referenced this issue Jan 9, 2021
@tkowalcz
Re #7: Cleaned up import
86d2dc4
tkowalcz added a commit that referenced this issue Jan 9, 2021
@tkowalcz
Re #7: Added component named MonitoringModule with empty implementati…
64626a3 
…on and one that uses AtomicLongs for counting events and Agrona's DistinctErrorLog for storing exceptions. While various counters are being updated by other components and response times are logged we do not provide way of plugging in ones own implementation or reading the counters back (yet).
tkowalcz added a commit that referenced this issue Jan 16, 2021
@tkowalcz
Re #7: Added configuration option 'maxRequestsInFlight' that is used …
a6e63cb 
…to size monitoring array. Added Netty pipeline stage for tracking number of requests in flight. Added monitoring of number of bytes of logs sent. Added thread that dumps stats to stdout.
tkowalcz added a commit that referenced this issue Jan 16, 2021
@tkowalcz
Re #7: Added default value to maxRequestsInFlight, and check in the b…
6b0ec58 
…uilder if it is > 0 (or else exception is thrown).
tkowalcz added a commit that referenced this issue Jan 16, 2021
@tkowalcz
Re #7: Cleaned up after moving debug thread logic to separate class.
90fcdfd
tkowalcz added a commit that referenced this issue Jan 22, 2021
@tkowalcz
Re #7: Added more tests and some small changes of monitoring and Nett…
67a0bfb 
…y code. Added closing connection if keepalive is not in force.
@tkowalcz tkowalcz linked a pull request Jan 22, 2021 that will close this issue
7 monitoring #20
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
core Relates to Core component
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

7 monitoring tkowalcz/tjahzi
1 participant
@tkowalcz