[MAPREDUCE-5190] Unnecessary condition test in RandomSampler

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.6.0
• Fix Version/s: 2.8.0, 3.0.0-alpha1
• Component/s: mrv2
• Hadoop Flags: Reviewed
• Assignee: Jingguo Yao
• Reporter: Jingguo Yao
• Created: 27/Apr/13 14:06
• Updated: 30/Aug/16 01:20
• Resolved: 22/Mar/15 04:35

Description

In getSampe method, there is a condition test after "int ind = r.nextInt(numSamples);". The test is "(ind != numSamples)".
This test is unneeded since nextInt(numSamples) will not return numSamples.