[KAFKA-1818] Code cleanup in ReplicationUtils including unit test

• Type: Improvement
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 0.8.1.1
• Fix Version/s: 0.9.0.0
• Component/s: replication
• Assignee: Eric Olander
• Reporter: Eric Olander
• Created: 14/Dec/14 19:01
• Updated: 23/Nov/15 03:30
• Resolved: 26/Jan/15 03:29

Description

Code in getLeaderIsrAndEpochForPartition() and parseLeaderAndIsr() was essentially reimplementing the flatMap function on the Option type. The attached patch refactors that code to more idiomatic Scala and provides a unit test over the affected code.