-
Notifications
You must be signed in to change notification settings - Fork 17.9k
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
runtime/debug: provide channel statistics #5611
Comments
Thank you for the feature request @MichaelTJones! @mknyszek, perhaps this issue might interest you? |
I don't think we're going to add something like this to existing packages, however there is #37112 for which a statistic like this would be appropriate. I think in practice for this to not be terribly slow we would need to do something similar to what we do for allocations: accumulate the statistics per-P and flush them during a stats-reading STW operation. |
Understood. This zombie proposal has staggered along for seven years and
can now die and find peace.
On Mon, Jun 8, 2020 at 8:26 AM Michael Knyszek ***@***.***> wrote:
I don't think we're going to add something like this to existing packages,
however there is #37112 <#37112> for
which a statistic like this would be appropriate.
I think in practice for this to not be terribly slow we would need to do
something similar to what we do for allocations: accumulate the statistics
per-P and flush them during a stats-reading STW operation.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#5611 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB4DFJOTTKSMMBSDJIVSR6DRVT7LPANCNFSM4NWLJYEQ>
.
--
*Michael T. Jonesmichael.jones@gmail.com <michael.jones@gmail.com>*
|
Thank you for the responses @mknyszek and @MichaelTJones!
Bits to bits, atoms to atoms, I shall thus close this issue. Thank you. |
The text was updated successfully, but these errors were encountered: