-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Refactor diags and write into internal database #2081
Conversation
This is a significant refactor that splits out the common diags code into its own file, allowing it to be used by other types of nodes (brokers). The actual code is mostly the same, but diags types (as well as the This allows historical monitoring of all diagnostics. Of particular interest will be GC performance, memory usage, number of shards and shard groups on the cluster, index points, amongst other important data |
763fa56
to
fa990e2
Compare
For example, here is a memory query from the internal database:
|
e375075
to
c156eaf
Compare
This will make it easier to use these common diagnostics function in other components, like the Broker, as well as minimizing growth of the server type, in terms of SLOC. This new diags code can be converted to InfluxQL rows, for easy display and re-writes back to the database for self-monitoring.
This allows historical monitoring of all diagnostics. Of particular interest will be GC performance, memory usage, number of shards and shard groups on the cluster, index points, amongst other important data.
Also, some fixes to casts. This may need further work.
👍 so awesome. |
Refactor diags and write into internal database
No description provided.