[HBASE-15052] Use EnvironmentEdgeManager in ReplicationSource

• Type: Bug
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 1.2.0, 1.1.2, 1.0.3, 0.98.16.1, 2.0.0
• Fix Version/s: 1.2.0, 1.0.3, 1.1.3, 0.98.17, 2.0.0
• Component/s: Replication
• Assignee: Matteo Bertozzi
• Reporter: Matteo Bertozzi
• Created: 30/Dec/15 00:23
• Updated: 08/Feb/16 04:57
• Resolved: 11/Jan/16 18:03

Description

ReplicationSource is passing System.currentTimeMillis() to MetricsSource.setAgeOfLastShippedOp() which is subtracting that from EnvironmentEdgeManager.currentTime().

// 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;

we should just use EnvironmentEdgeManager.currentTime() in ReplicationSource