Github user mccheah commented on a diff in the pull request:
https://github.com/apache/spark/pull/20553#discussion_r178371345
--- Diff: docs/running-on-kubernetes.md ---
@@ -576,14 +576,21 @@ specific to Spark on Kubernetes.
<td><code>spark.kubernetes.driver.limit.cores</code></td>
<td>(none)</td>
<td>
- Specify the hard CPU
[limit](https://kubernetes.io/docs/concepts/configuration/manage-compute-resources-container/#resource-requests-and-limits-of-pod-and-container)
for the driver pod.
+ Specify a hard cpu
[limit](https://kubernetes.io/docs/concepts/configuration/manage-compute-resources-container/#resource-requests-and-limits-of-pod-and-container)
for the driver pod.
+ </td>
+</tr>
+<tr>
+ <td><code>spark.kubernetes.executor.cores</code></td>
+ <td>(none)</td>
+ <td>
+ Specify the cpu request for each executor pod. Values conform to the
Kubernetes
[convention](https://kubernetes.io/docs/concepts/configuration/manage-compute-resources-container/#meaning-of-cpu).
Takes precendence over <code>spark.executor.cores</code> if set.
--- End diff --
Precedence in what sense? This won't override `spark.executor.cores` when
it comes to the number of concurrently running tasks. Think it's better to say
that it's distinct from spark.executor.cores.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
