[YARN-5992] revert the visibility of interface AllocationFileLoaderService.Listener to public for outside usage

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Duplicate
• Affects Version/s: 3.0.0-alpha2
• Component/s: scheduler
• Assignee: Unassigned
• Reporter: Pan Yuxuan
• Created: 12/Dec/16 09:53
• Updated: 27/Dec/16 22:35
• Resolved: 27/Dec/16 22:35

Description

The visibility of interface AllocationFileLoaderService.Listener is changed from public to default by YARN-4997.
This may cause some downsteam like Hive with compile error:

Hive/src/shims/scheduler/src/main/java/org/apache/hadoop/hive/schshim/FairSchedulerShim.java:[45,67] 
org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.AllocationFileLoaderService.Listener is not public in 
org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.AllocationFileLoaderService; cannot be accessed from outside package

org/apache/hadoop/hive/schshim/FairSchedulerShim.java uses the interface like below:

allocsLoader.setReloadListener(new AllocationFileLoaderService.Listener() {
      @Override
      public void onReload(AllocationConfiguration allocs) {
        allocConf.set(allocs);
      }
    });

So can we revert the visibility of AllocationFileLoaderService.Listener to public so that the downstream can use the interface as before, otherwise, the downstream should make some changes.

Issue Links

• duplicates: YARN-6000 Make AllocationFileLoaderService.Listener public
• is duplicated by: YARN-6000 Make AllocationFileLoaderService.Listener public