[AVRO-607] SpecificData.getSchema not thread-safe

• Type: Bug
• Status: Resolved
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: 1.3.3, 1.8.1
• Fix Version/s: 1.8.2
• Component/s: java
• Labels: newbie patch
• Release Note: Method org.apache.avro.specific.SpecificData::getSchema(type) to use Guava's LoadingCache instead of a Map to avoid race conditions.
• Assignee: Andrius Druzinis-Vitkus
• Reporter: Stephen Tu
• Created: 05/Aug/10 04:28
• Updated: 04/Sep/16 19:25
• Resolved: 04/Sep/16 00:20

Description

SpecificData.getSchema uses a WeakHashMap to cache schemas, but WeakHashMap is not thread-safe, and the method itself is not synchronized. Seems like this could lead to the data structure getting corrupted.

Issue Links

• blocks: AVRO-1885 Release 1.8.2
• links to: AVRO-607.patch
• links to: GitHub Pull Request #664