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

tracing: Routing of metrics. #3855

Closed
jutaro opened this issue May 11, 2022 · 1 comment · Fixed by #3876
Closed

tracing: Routing of metrics. #3855

jutaro opened this issue May 11, 2022 · 1 comment · Fixed by #3876
Assignees
@jutaro

Comments

@jutaro
Copy link
Contributor

jutaro commented May 11, 2022

No description provided.

@jutaro
Copy link
Contributor Author

jutaro commented May 11, 2022

Shall metrics be handled in a way as other trace messages, that they can be filtered according to the configuration, or shall they be displayed always? The first variant has the advantage that the computation of costly metrics can be avoided based on configuration. The second may be more in tune to what users expect.

@jutaro jutaro changed the title tracing: What about routing of metrics. Shall metrics be handled in a way as other trace messages, that they can be filtered according to the configuration, or shall they be diaplayed always? tracing: Routing of metrics. May 11, 2022
@jutaro jutaro self-assigned this May 12, 2022
@jutaro jutaro moved this from Todo to In Progress in Performance & Tracing May 19, 2022
@jutaro jutaro linked a pull request May 19, 2022 that will close this issue
trace-dispatcher: seperate routing of metrics #3876
Merged
@jutaro jutaro moved this from In Progress to In Review in Performance & Tracing May 19, 2022
@iohk-bors iohk-bors bot closed this as completed in #3876 May 19, 2022
Repository owner moved this from In Review to Done in Performance & Tracing May 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jutaro jutaro

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

trace-dispatcher: seperate routing of metrics IntersectMBO/cardano-node
2 participants
@deepfire @jutaro