GitHub user NicoK opened a pull request:

    https://github.com/apache/flink/pull/3056

    [FLINK-3150] make YARN container invocation configurable

    By using the `yarn.container-start-command-template` configuration 
parameter, the Flink start command can be altered/extended. By default, it is 
set to `"%java% %jvmmem% %jvmopts% %logging% %class% %args% %redirects%"` with
    - `java` = path to the Java executable
    - `jvmmem` = JVM memory limits and tweaks
    - `jvmopts` = misc options for the Java VM
    - `logging` = logging-related configuration settings
    - `class` = main class to execute
    - `args` = arguments for the main class
    - `redirects` = output redirects
    
    @rmetzger this is based on your previous pull request 
(https://github.com/apache/flink/pull/1446/files), can you have a look?

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/NicoK/flink FLINK-3150

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/flink/pull/3056.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #3056
    
----
commit f36599778d5e28401d68bbdd39d0c55e2eee3d0b
Author: Nico Kruber <n...@data-artisans.com>
Date:   2017-01-02T15:17:09Z

    [hotfix] fix YARN TM krb5 conf only available when logging enabled

commit f112d0c46d205623712fd8cda0c798152150dfb1
Author: Nico Kruber <n...@data-artisans.com>
Date:   2017-01-02T16:44:38Z

    [FLINK-3150] add a test for BootstrapTools#getTaskManagerShellCommand

commit deb04f971764a675b5e39a5609465f0179fa8d02
Author: Nico Kruber <n...@data-artisans.com>
Date:   2017-01-03T10:53:23Z

    [FLINK-3150] add a test for 
AbstractYarnClusterDescriptor#setupApplicationMasterContainer

commit a140f34ebe728c77f8afc051b3dda0066a252bc6
Author: Nico Kruber <n...@data-artisans.com>
Date:   2017-01-02T16:01:21Z

    [FLINK-3150] make YARN container invocation configurable
    
    By using the `yarn.container-start-command-template` configuration 
parameter,
    the Flink start command can be altered/extended. By default, it is set to
    `"%java% %jvmmem% %jvmopts% %logging% %class% %args% %redirects%"` with
    - `java` = path to the Java executable
    - `jvmmem` = JVM memory limits and tweaks
    - `jvmopts` = misc options for the Java VM
    - `logging` = logging-related configuration settings
    - `class` = main class to execute
    - `args` = arguments for the main class
    - `redirects` = output redirects
    
    This is based on a previous pull request and commits by @rmetzger, i.e.
    https://github.com/apache/flink/pull/1446/files

commit b17d563a626da17c82476a4b76fd62da325e4396
Author: Nico Kruber <n...@data-artisans.com>
Date:   2017-01-02T17:26:31Z

    [FLINK-3150] fe-factor BootstrapToolsTest#testGetTaskManagerShellCommand
    
    -> reduce duplication of strings for easier test changes

commit 6ff6724c2935956a0deea51286eb0930652a165f
Author: Nico Kruber <n...@data-artisans.com>
Date:   2017-01-02T17:27:33Z

    [FLINK-3150] extend BootstrapToolsTest#testGetTaskManagerShellCommand
    
    This now also tests some variants changing the new
    `yarn.container-start-command-template` configuration parameter.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to