[ https://issues.apache.org/jira/browse/HBASE-15052?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15092872#comment-15092872 ]
Hudson commented on HBASE-15052: -------------------------------- SUCCESS: Integrated in HBase-1.3 #490 (See [https://builds.apache.org/job/HBase-1.3/490/]) HBASE-15052 Use EnvironmentEdgeManager in ReplicationSource (matteo.bertozzi: rev bb5026c6a42912334bb08cc65e983e80ee49ba06) * hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSource.java > Use EnvironmentEdgeManager in ReplicationSource > ------------------------------------------------ > > Key: HBASE-15052 > URL: https://issues.apache.org/jira/browse/HBASE-15052 > Project: HBase > Issue Type: Bug > Components: Replication > Affects Versions: 2.0.0, 1.2.0, 1.1.2, 1.0.3, 0.98.16.1 > Reporter: Matteo Bertozzi > Assignee: Matteo Bertozzi > Priority: Trivial > Fix For: 2.0.0, 1.2.1, 1.1.3, 0.98.17, 1.0.4 > > Attachments: HBASE-15052-v0.patch, HBASE-15052-v00.patch > > > ReplicationSource is passing System.currentTimeMillis() to > MetricsSource.setAgeOfLastShippedOp() which is subtracting that from > EnvironmentEdgeManager.currentTime(). > {code} > // if there was nothing to ship and it's not an error > // set "ageOfLastShippedOp" to <now> to indicate that we're current > metrics.setAgeOfLastShippedOp(System.currentTimeMillis(), walGroupId); > public void setAgeOfLastShippedOp(long timestamp, String walGroup) { > long age = EnvironmentEdgeManager.currentTime() - timestamp; > {code} > we should just use EnvironmentEdgeManager.currentTime() in ReplicationSource -- This message was sent by Atlassian JIRA (v6.3.4#6332)