[THRIFT-675] Generated findByValue() should just switch on value instead of mapping

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.2
• Fix Version/s: 0.3
• Component/s: Java - Compiler
• Assignee: Bryan Duxbury
• Reporter: Chad Walters
• Created: 14/Jan/10 05:16
• Updated: 01/Nov/11 02:51
• Resolved: 16/Jan/10 09:14

Description

Generated enum classes currently use a map in findByValue() to go from an integer value to an enum.

Instead of using a map, it is faster to simply switch on the value with a default case returning null.

On some benchmarks I ran, this saved 7% of deserialization time.