Github user foxish commented on a diff in the pull request:
https://github.com/apache/spark/pull/20553#discussion_r170137631
--- 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
[limit](https://kubernetes.io/docs/concepts/configuration/manage-compute-resources-container/#resource-requests-and-limits-of-pod-and-container)
on the amount of CPU cores for the driver pod.
--- End diff --
I think it reads better without "the amount of", i.e. "Specify a hard limit
on CPU cores for the driver pod". Same comment for the below section as well.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
