Skip to content

[WIP] Allocation metrics#542

Draft
r1viollet wants to merge 1 commit intomainfrom
r1viollet/alloc_metrics
Draft

[WIP] Allocation metrics#542
r1viollet wants to merge 1 commit intomainfrom
r1viollet/alloc_metrics

Conversation

@r1viollet
Copy link
Copy Markdown
Collaborator

Prototype upload of metrics in separate json

What does this PR do?

A brief description of the change being made with this pull request.

Motivation

What inspired you to submit this pull request? In order to motivate your point, it may be valuable to explain why this change is useful or what problem is being solved.

Additional Notes

This is the section to put technical guidance/constraints, call out potential regressions, cite sources, and in general offer some exposition on the how of the PR. This section doesn't need to be incredibly detailed, but it makes life easier for reviewers!

How to test the change?

Describe here in detail how the change can be validated. This is a great section to call out specific tests you've added or improved, or to acknowledge code sections which are particularly difficult to test.

Prototype upload of metrics in separate json
Comment thread src/ddprof_worker.cc
}
}
// Allocation tracker stats are per watcher, so always accumulate
total_arena_count += pid_stacks._active_shards;
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor: slightly unexpected if we start adding these...

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

Successfully merging this pull request may close these issues.

1 participant