[GIRAPH-105] BspServiceMaster.checkWorkers() should return empty lists instead of null

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 0.1.0
• Fix Version/s: 0.1.0
• Assignee: Sebastian Schelter
• Reporter: Sebastian Schelter
• Created: 17/Dec/11 11:13
• Updated: 31/Jan/12 23:26
• Resolved: 18/Dec/11 17:10

Description

BspServiceMaster.checkWorkers() is invoked in BspServiceMaster.coordinateSuperstep() and in BspServiceMaster.createInputSplits(). Both check for an empty list to fail the job in case something has gone wrong. However, checkWorkers() returns null in case of problems, causing an NPE in the calling code.