[SPARK-19006][DOCS]mention spark.kryoserializer.buffer.max must be less than 2048m in doc #16412

Closed
wants to merge 1 commit into
from

Projects

None yet

3 participants

@cnZach
cnZach commented Dec 27, 2016

What changes were proposed in this pull request?

On configuration doc page:https://spark.apache.org/docs/latest/configuration.html
We mentioned spark.kryoserializer.buffer.max : Maximum allowable size of Kryo serialization buffer. This must be larger than any object you attempt to serialize. Increase this if you get a "buffer limit exceeded" exception inside Kryo.
from source code, it has hard coded upper limit :

val maxBufferSizeMb = conf.getSizeAsMb("spark.kryoserializer.buffer.max", "64m").toInt
if (maxBufferSizeMb >= ByteUnit.GiB.toMiB(2))
{ throw new IllegalArgumentException("spark.kryoserializer.buffer.max must be less than " + s"2048 mb, got: + $maxBufferSizeMb mb.") }

We should mention "this value must be less than 2048 mb" on the configuration doc page as well.

How was this patch tested?

None. Since it's minor doc change.

@SparkQA
SparkQA commented Dec 27, 2016

Test build #3514 has finished for PR 16412 at commit ca3e142.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.
@srowen
Member
srowen commented Dec 27, 2016

Merged to master

@asfgit asfgit pushed a commit that closed this pull request Dec 27, 2016
Yuexin Zhang [SPARK-19006][DOCS] mention spark.kryoserializer.buffer.max must be l…
…ess than 2048m in doc

## What changes were proposed in this pull request?

On configuration doc page:https://spark.apache.org/docs/latest/configuration.html
We mentioned spark.kryoserializer.buffer.max : Maximum allowable size of Kryo serialization buffer. This must be larger than any object you attempt to serialize. Increase this if you get a "buffer limit exceeded" exception inside Kryo.
from source code, it has hard coded upper limit :
```
val maxBufferSizeMb = conf.getSizeAsMb("spark.kryoserializer.buffer.max", "64m").toInt
if (maxBufferSizeMb >= ByteUnit.GiB.toMiB(2))
{ throw new IllegalArgumentException("spark.kryoserializer.buffer.max must be less than " + s"2048 mb, got: + $maxBufferSizeMb mb.") }
```
We should mention "this value must be less than 2048 mb" on the configuration doc page as well.

## How was this patch tested?

None. Since it's minor doc change.

Author: Yuexin Zhang <[email protected]>

Closes #16412 from cnZach/SPARK-19006.
28ab0ec
@asfgit asfgit closed this in 28ab0ec Dec 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment