Mailing-List: contact issues-help@flink.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@flink.apache.org
Date: Thu, 27 Jul 2017 15:30:01 +0000 (UTC)
From: "ASF GitHub Bot (JIRA)" <jira@apache.org>
To: issues@flink.apache.org
Message-ID: <JIRA.13085430.1499411825000.27662.1501169401243@Atlassian.JIRA>
In-Reply-To: <JIRA.13085430.1499411825000@Atlassian.JIRA>
References: <JIRA.13085430.1499411825000@Atlassian.JIRA> <JIRA.13085430.1499411825379@jira-lw-us.apache.org>
Subject: [jira] [Commented] (FLINK-7125) Remove configuration loading from
 AbstractYarnClusterDescriptor
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Thu, 27 Jul 2017 15:30:08 -0000


    [ https://issues.apache.org/jira/browse/FLINK-7125?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16103345#comment-16103345 ] 

ASF GitHub Bot commented on FLINK-7125:
---------------------------------------

Github user tillrohrmann commented on a diff in the pull request:

    https://github.com/apache/flink/pull/4280#discussion_r129875590
  
    --- Diff: flink-yarn/src/main/java/org/apache/flink/yarn/cli/FlinkYarnCLI.java ---
    @@ -214,22 +218,21 @@ public void addGeneralOptions(Options baseOptions) {
     
     	@Override
     	public YarnClusterClientV2 retrieveCluster(
    -			CommandLine cmdLine,
    -			Configuration config) throws UnsupportedOperationException {
    +		CommandLine cmdLine,
    +		Configuration config, String configurationDirectory) throws UnsupportedOperationException {
    --- End diff --
    
    I shouldn't use automatic reformatting. It screws up the formatting ;-)


> Remove configuration loading from AbstractYarnClusterDescriptor
> ---------------------------------------------------------------
>
>                 Key: FLINK-7125
>                 URL: https://issues.apache.org/jira/browse/FLINK-7125
>             Project: Flink
>          Issue Type: Improvement
>          Components: YARN
>            Reporter: Till Rohrmann
>            Assignee: Till Rohrmann
>              Labels: flip-6
>
> We should remove the explicit configuration loading from the {{AbstractYarnClusterDescriptor}} and instead pass it to the class via the constructor. This passed in configuration should then be sent to the application master when being started. That way, we can easily configure the started Yarn cluster.


--
This message was sent by Atlassian JIRA
(v6.4.14#64029)