[ACCUMULO-4153] Change getCodec method in BCFile's Compression class

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.6.5
• Fix Version/s: 1.6.6, 1.7.3, 1.8.0
• Component/s: core
• Flags: Patch
• Assignee: Marc Parisi
• Reporter: Marc Parisi
• Created: 24/Feb/16 18:00
• Updated: 14/Jul/16 19:02
• Resolved: 08/Jul/16 19:37

Description

Seeing major blockages blockages on the Synchronized method of getCodec. My initial change was to change it to double checked locking; however, spoke with kturner and I'm changing this to an AtomicReference. Have a patch for the former but am in the process of testing the AtomicReference patch

Note this is for Gzip

Issue Links

• breaks: ACCUMULO-4372 CompressionTest unit test failure "SNAPPY created too many codecs expected"
• links to: GitHub Pull Request #73
• links to: GitHub Pull Request #105
• links to: GitHub Pull Request #106