[YARN-3794] TestRMEmbeddedElector fails because of ambiguous LOG reference

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.7.0
• Fix Version/s: 2.8.0, 3.0.0-alpha1
• Component/s: test
• Hadoop Flags: Reviewed
• Assignee: Chengbing Liu
• Reporter: Chengbing Liu
• Created: 11/Jun/15 05:51
• Updated: 30/Aug/16 01:20
• Resolved: 12/Jun/15 08:20

Description

After YARN-2921, MockRM has also a LOG field. Therefore LOG in the following code snippet is ambiguous.

    protected AdminService createAdminService() {
      return new AdminService(MockRMWithElector.this, getRMContext()) {
        @Override
        protected EmbeddedElectorService createEmbeddedElectorService() {
          return new EmbeddedElectorService(getRMContext()) {
            @Override
            public void becomeActive() throws
                ServiceFailedException {
              try {
                callbackCalled.set(true);
                LOG.info("Callback called. Sleeping now");
                Thread.sleep(delayMs);
                LOG.info("Sleep done");
              } catch (InterruptedException e) {
                e.printStackTrace();
              }
              super.becomeActive();
            }
          };
        }
      };
    }

Eclipse gives the following error:

The field LOG is defined in an inherited type and an enclosing scope

IMO, we should fix this as TestRMEmbeddedElector.LOG