[HBASE-13703] ReplicateContext should not be a member of ReplicationSource

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 0.98.13, 1.0.2, 1.2.0, 1.1.1, 2.0.0
• Hadoop Flags: Reviewed
• Assignee: Lars Hofhansl
• Reporter: Lars Hofhansl
• Created: 17/May/15 20:09
• Updated: 31/Aug/15 22:39
• Resolved: 22/May/15 05:10

Description

The ReplicateContext object is created once per ReplicationSource and then reused when we have something to ship to the sinks.

This is a misguided optimization. ReplicateContext is very lightweight (definitely compared to the all the work and copying the ReplicationSource is doing) and, crucially, it prevent the the entries array from being collected after it was successfully copied to the sink, wasting potentially a lot of heap.
The entries array itself holds reference to WAL entries on the heap, that now also cannot be collected.