[GitHub] spark pull request #20761: [SPARK-20327][CORE][YARN] Add CLI support for YAR...

[felixcheung]

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

    https://github.com/apache/spark/pull/20761#discussion_r227462444
  
    --- Diff: docs/running-on-yarn.md ---
    @@ -121,6 +121,43 @@ To use a custom metrics.properties for the application 
master and executors, upd
         Use lower-case suffixes, e.g. <code>k</code>, <code>m</code>, 
<code>g</code>, <code>t</code>, and <code>p</code>, for kibi-, mebi-, gibi-, 
tebi-, and pebibytes, respectively.
       </td>
     </tr>
    +<tr>
    +  <td><code>spark.yarn.am.resource.{resource-type}</code></td>
    +  <td><code>(none)</code></td>
    +  <td>
    +    Amount of resource to use for the YARN Application Master in client 
mode.
    +    In cluster mode, use 
<code>spark.yarn.driver.resource.&lt;resource-type&gt;</code> instead.
    --- End diff --
    
    nit: looks like `spark.yarn.driver.resource.&lt;resource-type&gt;` should 
be `spark.yarn.driver.resource.{resource-type}`
    
    (yes, I realize `resource-type` is to be replaced with)


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org