[ACCUMULO-760] Batch Writer implementation doesn't need to synchronize on addMutation(String, Iterator)

• Type: Improvement
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Fix Version/s: 1.5.0
• Component/s: client
• Labels: batch lock synchronized thread writer
• Assignee: Christopher Tubbs
• Reporter: Christopher Tubbs
• Created: 09/Sep/12 01:52
• Updated: 10/Sep/12 15:28
• Resolved: 10/Sep/12 15:28

Description

org.apache.accumulo.core.client.impl.TabletServerBatchWriter locks when a mutation is added. It does not need to lock when iterating through an Iterator<Mutation> to add them one at a time. This can have add significant performance issues to the other threads servicing the batch writer's bins if the Iterator that is passed in is poorly performing.