[YARN-746] rename Service.register() and Service.unregister() to registerServiceListener() & unregisterServiceListener() respectively

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.1.0-beta
• Hadoop Flags: Incompatible change, Reviewed
• Assignee: Steve Loughran
• Reporter: Steve Loughran
• Created: 01/Jun/13 11:06
• Updated: 18/Mar/15 18:30
• Resolved: 13/Jun/13 17:51

Description

make it clear what you are registering on a Service by naming the methods registerServiceListener() & unregisterServiceListener() respectively.

This only affects a couple of production classes; Service.register() and is used in some of the lifecycle tests of the YARN-530. There are no tests of Service.unregister(), which is something that could be corrected.

Issue Links

• is duplicated by: MAPREDUCE-3536 consider whether the same instance of a ServiceStateChangeListener should be allowed to listen to events
• is required by: YARN-678 Delete FilterService