[S4-74] NullPointerException on TCPEmitter.onChange()

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.5.0
• Fix Version/s: 0.5.0
• Assignee: Matthieu Morel
• Reporter: Daniel Gómez Ferro
• Created: 09/Jul/12 08:58
• Updated: 13/Jul/12 14:37
• Resolved: 13/Jul/12 14:37

Description

TCPEmitter is registering itself as a listener in its constructor: this.topology.addListener(this);

The topology is triggering a refresh before the TCPEmitter is fully constructed, so TCPEmitter.partitionNodeMap is still null during onChange() execution.

TCPEmitter shouldn't register itself as a listener until it's fully built, see http://www.ibm.com/developerworks/java/library/j-jtp0618/index.html#2 (Don't publish the "this" reference during construction)