Mailing-List: contact yarn-dev-help@hadoop.apache.org; run by ezmlm
Precedence: bulk
Reply-To: yarn-dev@hadoop.apache.org
Date: Sun, 2 Nov 2014 21:40:34 +0000 (UTC)
From: "zhihai xu (JIRA)" <jira@apache.org>
To: yarn-dev@hadoop.apache.org
Message-ID: <JIRA.12752338.1414964421000.395174.1414964434009@Atlassian.JIRA>
In-Reply-To: <JIRA.12752338.1414964421000@Atlassian.JIRA>
References: <JIRA.12752338.1414964421000@Atlassian.JIRA>
 <JIRA.12752338.1414964421331@arcas>
Subject: [jira] [Created] (YARN-2799) cleanup TestLogAggregationService
 based on the change in YARN-90
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

zhihai xu created YARN-2799:
-------------------------------

             Summary: cleanup TestLogAggregationService based on the change in YARN-90
                 Key: YARN-2799
                 URL: https://issues.apache.org/jira/browse/YARN-2799
             Project: Hadoop YARN
          Issue Type: Improvement
          Components: test
            Reporter: zhihai xu
            Priority: Minor


cleanup TestLogAggregationService based on the change in YARN-90.
The following code is added to setup in YARN-90, 
{code}
    dispatcher = createDispatcher();
    appEventHandler = mock(EventHandler.class);
    dispatcher.register(ApplicationEventType.class, appEventHandler);
{code}
In this case, we should remove all these code from each test function to avoid duplicate code.

Same for dispatcher.stop() which is in tearDown,
we can remove dispatcher.stop() from from each test function also because it will always be called from tearDown for each test.


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)