[GitHub] [flink] zentol commented on a change in pull request #8089: [FLINK-11985][DOC] Remove ignored command line parameter from yarn_setup.md

[GitBox <gi...@apache.org>]

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:
users@infra.apache.org


With regards,
Apache Git Services