srowen commented on a change in pull request #25689: [SPARK-28972][DOCS]
Describe unit for `spark.memory.offHeap.size`
URL: https://github.com/apache/spark/pull/25689#discussion_r324475755
##########
File path: core/src/main/scala/org/apache/spark/internal/config/package.scala
##########
@@ -243,7 +243,8 @@ package object config {
.createWithDefault(false)
private[spark] val MEMORY_OFFHEAP_SIZE =
ConfigBuilder("spark.memory.offHeap.size")
- .doc("The absolute amount of memory in bytes which can be used for
off-heap allocation. " +
+ .doc("The absolute amount of memory which can be used for off-heap
allocation. " +
+ "Default unit is bytes unless specifies otherwise. " +
Review comment:
Just write ", in bytes unless otherwise specified." for consistency
----------------------------------------------------------------
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]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]