flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (FLINK-7125) Remove configuration loading from AbstractYarnClusterDescriptor
Date Thu, 27 Jul 2017 15:30:01 GMT

    [ 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)

Mime
View raw message