Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/5448#discussion_r193221496
--- Diff: docs/_includes/generated/task_manager_configuration.html ---
@@ -84,13 +84,13 @@
</tr>
<tr>
<td><h5>taskmanager.memory.segment-size</h5></td>
- <td style="word-wrap: break-word;">32768</td>
- <td>Size of memory buffers used by the network stack and the
memory manager (in bytes).</td>
+ <td style="word-wrap: break-word;">"32768"</td>
+ <td>Size of memory buffers used by the network stack and the
memory manager.</td>
</tr>
<tr>
<td><h5>taskmanager.memory.size</h5></td>
- <td style="word-wrap: break-word;">-1</td>
- <td>Amount of memory to be allocated by the task manager's
memory manager (in megabytes). If not set, a relative fraction will be
allocated.</td>
+ <td style="word-wrap: break-word;">"-1"</td>
--- End diff --
looking at the
[parser](https://github.com/yanghua/flink/blob/39fd2efb66bd6fcac3f86e953729831a49bc7709/flink-core/src/main/java/org/apache/flink/configuration/MemorySize.java#L191)
it appears that `MemorySize` does not support negative values (rightfully so).
We may have to change the default to 0.
---
