agura commented on a change in pull request #8383:
URL: https://github.com/apache/ignite/pull/8383#discussion_r514519089
##########
File path: modules/core/src/main/java/org/apache/ignite/DataStorageMetrics.java
##########
@@ -98,6 +98,13 @@
*/
public long getLastCheckpointDuration();
+ /**
+ * Start timestamp of the last checkpoint
Review comment:
Please fix javadoc. It should describe what this method does. It is
obviously that this method doesn't start anything. So correct comment should
start with "Returns" and describe what exactly it returns. Also each sentence
in javadoc must end with dot (.). Please fix it here and check the rest comment
in your change.
I believe also that time instead of timestamp is better word for an
end-user. Again, here and in other changed lines.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
