dongjoon-hyun commented on a change in pull request #32610: URL: https://github.com/apache/spark/pull/32610#discussion_r648001244
########## File path: docs/running-on-kubernetes.md ########## @@ -889,7 +889,10 @@ See the [configuration page](configuration.html) for information on Spark config <td><code>spark.kubernetes.executor.podNamePrefix</code></td> <td>(none)</td> <td> - Prefix to use in front of the executor pod names. + Prefix to use in front of the executor pod names. It must conform the rules defined by the Kubernetes + <a href="https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#dns-label-names";>DNS Label Names</a>. + Besides, the executor pod names will be generated in the form of <code>$podNamePrefix-exec-$id</code>, where the `id` is + a positive long value, so the length of the `podNamePrefix` need to be less than or equal to 38(= 63 - 19 - 6). Review comment: `a positive long value` looks wrong to me. It's `Int`, isn't it? -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
