-
Notifications
You must be signed in to change notification settings - Fork 387
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
[Cassandra] Add metric_type mapping for the fields of metrics
datastream.
#6858
[Cassandra] Add metric_type mapping for the fields of metrics
datastream.
#6858
Conversation
🌐 Coverage report
|
metrics
datastream.
- name: pending | ||
description: compaction pending tasks. | ||
type: long | ||
metric_type: gauge | ||
- name: dropped_message |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we are not sure that these fields are counter then can we map it as a gauge?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These metrics fall into either the Timer or Meter category, so didn't map it to gauge.
https://cassandra.apache.org/doc/latest/cassandra/operating/metrics.html#dropped-metrics
@lalit-satapathy thoughts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Which metric here is of doubt?
@kush-elastic any pending reviews?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Raised an issue to take care of these fields when there is support for metric_type timer
and meter
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After conducting a thorough analysis, I have assigned the metric_type "counter" to these fields. This decision was made based on the fact that there are multiple attributes related to dropped messages, and our pipeline specifically processes the "Count" attribute for all types of dropped messages. It is important to note that the "Count" attribute itself is of type counter.
For further clarification and better understanding, please refer to the link and screenshots.
![Screenshot 2023-07-19 at 12 50 44 PM](https://private-user-images.githubusercontent.com/102972658/254495341-f60d7474-bb0d-4e68-9a48-e6f49861f96b.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTk2NzAwMDgsIm5iZiI6MTcxOTY2OTcwOCwicGF0aCI6Ii8xMDI5NzI2NTgvMjU0NDk1MzQxLWY2MGQ3NDc0LWJiMGQtNGU2OC05YTQ4LWU2ZjQ5ODYxZjk2Yi5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNjI5JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDYyOVQxNDAxNDhaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT01OTAyZjBiNDllZDQxMjc1OWEyYzZlNGEzZGI3NjZjNzE4OThhMzUzYWU4MmRjMTZlZTAxOGYzNzc0NTA5Yjc5JlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.zOjMD3_MnoXpfqEFG2EePON0twK543-WWlPIyRR8Gf0)
![Screenshot 2023-07-19 at 12 51 13 PM](https://private-user-images.githubusercontent.com/102972658/254495401-57846e9c-b01c-41cd-ac93-76d5eb2651c0.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTk2NzAwMDgsIm5iZiI6MTcxOTY2OTcwOCwicGF0aCI6Ii8xMDI5NzI2NTgvMjU0NDk1NDAxLTU3ODQ2ZTljLWIwMWMtNDFjZC1hYzkzLTc2ZDVlYjI2NTFjMC5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNjI5JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDYyOVQxNDAxNDhaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1jY2Q5MTQ4OTMwZTBjY2Q3NTk1YzRlZDE5Y2M4ZmQ5MWNkOTcwODljMTAyMmQxYTE3NjRhM2Q1Yjc0YmIyZDNlJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.zyI5Myg4bytYNAVETYgpFFz_kGVSO6UIinePvdKqnB0)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@agithomas, Can you confirm if this mapping is correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, as Richa as validated with Cassandra documentation
Package cassandra - 1.8.1 containing this change is available at https://epr.elastic.co/search?package=cassandra |
…tream. (#6858) * [Cassandra] Add metric_type mapping for the fields of datastream.
What does this PR do?
This PR adds metric_type mapping for the fields of
metrics
datastream.Checklist
changelog.yml
file.Related issues