You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ignite.apache.org by "Denis Mekhanikov (Jira)" <ji...@apache.org> on 2021/10/10 13:17:00 UTC

[jira] [Commented] (IGNITE-13631) Improve names and descriptions of data storage metrics

    [ https://issues.apache.org/jira/browse/IGNITE-13631?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17426808#comment-17426808 ] 

Denis Mekhanikov commented on IGNITE-13631:
-------------------------------------------

[~nizhikov], thanks for the review! I fixed your comments.

Could you help with a merge?

> Improve names and descriptions of data storage metrics
> ------------------------------------------------------
>
>                 Key: IGNITE-13631
>                 URL: https://issues.apache.org/jira/browse/IGNITE-13631
>             Project: Ignite
>          Issue Type: Improvement
>            Reporter: Denis Mekhanikov
>            Assignee: Denis Mekhanikov
>            Priority: Major
>          Time Spent: 50m
>  Remaining Estimate: 0h
>
> Data storage metrics have unclear descriptions. They need to be improved.
> ||Metric||Description||Comment||
> |*WalLoggingRate*|Average number of WAL records per second written during the last time interval.|The "time interval" part is unclear. Which time interval?|
> |*WalFsyncTimeDuration*|Total duration of fsync|Why not just *WalFsyncDuration*?
> The description could be more verbose.|
> |*WalFsyncTimeNum*|Total count of fsync|Why not just *WalFsyncNum*? The description could be more verbose|
> |*WalBuffPollSpinsRate*|WAL buffer poll spins number over the last time interval. |Over which time interval?|
> |*LastCheckpointMarkDuration*|Duration of the checkpoint lock wait in milliseconds |The description doesn't match the name.|
> |*CheckpointTotalTime*|Total duration of checkpoint|Is it the duration of the last checkpoint or all checkpoints from the beginning?|
> |*StorageSize*|Storage space allocated, in bytes.|It's unclear which storage this is about. Is disk included, or is it just about memory?|
> |*WalTotalSize*|Total size in bytes for storage wal files.|WAL should be capital. The grammar|



--
This message was sent by Atlassian Jira
(v8.3.4#803005)