Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/4512#discussion_r25133423
--- Diff: docs/configuration.md ---
@@ -882,36 +882,24 @@ Apart from these, the following properties are also
available, and may be useful
<td><code>spark.akka.heartbeat.pauses</code></td>
<td>6000</td>
<td>
- This is set to a larger value to disable failure detector that comes
inbuilt akka. It can be
- enabled again, if you plan to use this feature (Not recommended).
Acceptable heart beat pause
- in seconds for akka. This can be used to control sensitivity to gc
pauses. Tune this in
- combination of `spark.akka.heartbeat.interval` and
`spark.akka.failure-detector.threshold`
- if you need to.
- </td>
-</tr>
-<tr>
- <td><code>spark.akka.failure-detector.threshold</code></td>
- <td>300.0</td>
- <td>
- This is set to a larger value to disable failure detector that comes
inbuilt akka. It can be
- enabled again, if you plan to use this feature (Not recommended).
This maps to akka's
- `akka.remote.transport-failure-detector.threshold`. Tune this in
combination of
- `spark.akka.heartbeat.pauses` and `spark.akka.heartbeat.interval` if
you need to.
+ This is set to a larger value to disable transport failure detector
that comes inbuilt akka.
--- End diff --
Let's fix the grammar while we're here. "disable transport failure" ->
"disable the transport failure". "inbuilt akka" -> "built in to Akka". "akka"
-> "Akka" when not in a property name. "in combination of" -> "along with".
"gc" -> "GC".
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
