[GitHub] flink pull request #5392: [FLINK-8475][config][docs] Integrate JM options

[asfgit]

Github user asfgit closed the pull request at:

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


---

[GitHub] flink pull request #5392: [FLINK-8475][config][docs] Integrate JM options

[zentol]

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

https://github.com/apache/flink/pull/5392#discussion_r167596687
  
--- Diff: docs/ops/config.md ---
@@ -322,7 +321,6 @@ The following parameters configure Flink's JobManager 
and TaskManagers.
 
 - `taskmanager.exit-on-fatal-akka-error`: Whether the TaskManager shall be 
terminated in case of a fatal Akka error (quarantining event). (DEFAULT: 
**false**)
 
-- `jobmanager.tdd.offload.minsize`: Maximum size of the 
`TaskDeploymentDescriptor`'s serialized task and job information to still 
transmit them via RPC. Larger blobs may be offloaded to the BLOB server. 
(DEFAULT: **1 KiB**).
--- End diff --

double-checked and yes, I couldn't find a reference anywhere.


---

[GitHub] flink pull request #5392: [FLINK-8475][config][docs] Integrate JM options

[aljoscha]

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

https://github.com/apache/flink/pull/5392#discussion_r167594880
  
--- Diff: docs/ops/config.md ---
@@ -322,7 +321,6 @@ The following parameters configure Flink's JobManager 
and TaskManagers.
 
 - `taskmanager.exit-on-fatal-akka-error`: Whether the TaskManager shall be 
terminated in case of a fatal Akka error (quarantining event). (DEFAULT: 
**false**)
 
-- `jobmanager.tdd.offload.minsize`: Maximum size of the 
`TaskDeploymentDescriptor`'s serialized task and job information to still 
transmit them via RPC. Larger blobs may be offloaded to the BLOB server. 
(DEFAULT: **1 KiB**).
--- End diff --

This setting simply doesn't exist anymore?


---

[GitHub] flink pull request #5392: [FLINK-8475][config][docs] Integrate JM options

[zentol]

GitHub user zentol opened a pull request:

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

[FLINK-8475][config][docs] Integrate JM options

## What is the purpose of the change

This PR integrates the JobManager `ConfigOptions` into the configuration 
docs generator.

## Brief change log

* Add missing descriptions to config options (derived from existing 
description/javadocs)
* integrate jobmanager configuration table into `config.md` and separate 
job- and taskmanager sections

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

$ git pull https://github.com/zentol/flink 8475_jm

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

https://github.com/apache/flink/pull/5392.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 #5392


commit 61bb0eff965cd066edebc5e4a167dd9bd3a07f77
Author: zentol 
Date:   2018-01-22T16:32:38Z

[FLINK-8475][config][docs] Integrate JM options




---