[HBASE-17652] Use 'default' keyword in Coprocessor Observer interfaces to get rid of 'Base...Observer' implementations.

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Duplicate
• Component/s: Coprocessors
• Labels: coprocessors incompatible
• Assignee: Apekshit Sharma
• Reporter: Apekshit Sharma
• Created: 15/Feb/17 22:27
• Updated: 26/Aug/17 02:06
• Resolved: 25/Feb/17 10:51

Description

In cases where one might need to use multiple observers, say region, master and regionserver; and the fact that only one class can be extended, it gives rise to following pattern:

    public class BaseMasterAndRegionObserver
      extends BaseRegionObserver
      implements MasterObserver

    class AccessController
      extends BaseMasterAndRegionObserver
      implements RegionServerObserver

were BaseMasterAndRegionObserver is full copy of BaseMasterObserver.

There is an example of simple case too where the current design fails.
Say only one observer is needed by the coprocessor, but the design doesn't permit extending even that single observer (see RSGroupAdminEndpoint), that leads to full copy of Base...Observer class into coprocessor class leading to 1000s of lines of code and this ugly mix of 5 main functions with 100 useless functions.

Issue Links

• duplicates: HBASE-17312 [JDK8] Use default method for Observer Coprocessors
• links to: Review Board (master)