Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21454#discussion_r191625505
--- Diff: core/src/main/scala/org/apache/spark/SparkConf.scala ---
@@ -265,108 +265,121 @@ class SparkConf(loadDefaults: Boolean) extends
Cloneable with Logging with Seria
* Get a time parameter as seconds; throws a NoSuchElementException if
it's not set. If no
* suffix is provided then seconds are assumed.
* @throws java.util.NoSuchElementException If the time parameter is not
set
+ * @throws IllegalArgumentException If the value can't be interpreted as
seconds
--- End diff --
I usually avoid abbreviation in the documentation though. `can't` ->
`cannot`.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
