[
https://issues.apache.org/jira/browse/HDFS-5591?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13956769#comment-13956769
]
Charles Lamb commented on HDFS-5591:
------------------------------------
You're right. But wouldn't Time.mNow be > than lastCheckpointTime
meaning that it should be
Time.monotonicNow() - lastCheckpointTime instead of the opposite?
> Checkpointing should use monotonic time when calculating period
> ---------------------------------------------------------------
>
> Key: HDFS-5591
> URL: https://issues.apache.org/jira/browse/HDFS-5591
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: namenode
> Affects Versions: 2.2.0
> Reporter: Andrew Wang
> Assignee: Charles Lamb
> Priority: Minor
> Labels: newbie
> Attachments: HDFS-5591.001.patch, HDFS-5591.002.patch,
> HDFS-5591.003.patch, HDFS-5591.004.patch
>
>
> Both StandbyCheckpointer and SecondaryNameNode use {{Time.now}} rather than
> {{Time.monotonicNow}} to calculate how long it's been since the last
> checkpoint. This can lead to issues when the system time is changed.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
