zentol commented on a change in pull request #8089: [FLINK-11985][DOC] Remove
ignored command line parameter from yarn_setup.md
URL: https://github.com/apache/flink/pull/8089#discussion_r284601483
##########
File path: docs/ops/deployment/yarn_setup.md
##########
@@ -96,8 +96,6 @@ This command will show you the following overview:
{% highlight bash %}
Usage:
- Required
- -n,--container <arg> Number of YARN container to allocate (=Number of
Task Managers)
Review comment:
This listing was generated by the CLI; removing it doesn't make sense if the
next guy could just be adding it back.
Please change the Option description in the `FlinkYarnSessionCli` to
explicitly refer to this as a deprecated legacy option, and update this section
with the CLI help output..
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
