Skip to content

Conversation

@zzcclp
Copy link
Contributor

@zzcclp zzcclp commented Jan 7, 2016

modify 'spark.memory.offHeap.enabled' default value to false

@zzcclp
Copy link
Contributor Author

zzcclp commented Jan 7, 2016

@JoshRosen

@sarutak
Copy link
Member

sarutak commented Jan 7, 2016

ok to test.

@sarutak
Copy link
Member

sarutak commented Jan 7, 2016

LGTM

@sarutak
Copy link
Member

sarutak commented Jan 7, 2016

@zzcclp Could you file this issue in JIRA and modify the title of this PR like [SPARK-XXXX][DOC]fix... ?

@zzcclp
Copy link
Contributor Author

zzcclp commented Jan 7, 2016

@sarutak , i think this is a little fix, dose it need to file a issue in JIRA?

@rxin
Copy link
Contributor

rxin commented Jan 7, 2016

For this one it is probably OK. We can just merge it (into master & branch-1.6).

@rxin
Copy link
Contributor

rxin commented Jan 7, 2016

I've merged this.

@asfgit asfgit closed this in 84e77a1 Jan 7, 2016
asfgit pushed a commit that referenced this pull request Jan 7, 2016
modify 'spark.memory.offHeap.enabled' default value to false

Author: zzcclp <[email protected]>

Closes #10633 from zzcclp/fix_spark.memory.offHeap.enabled_default_value.

(cherry picked from commit 84e77a1)
Signed-off-by: Reynold Xin <[email protected]>
@zzcclp
Copy link
Contributor Author

zzcclp commented Jan 7, 2016

@sarutak , @rxin , thanks

@AmplabJenkins
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48910/
Test FAILed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants