Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use SparkConf object to configure SparkContext #223

Merged
merged 1 commit into from
Apr 23, 2014

Conversation

massie
Copy link
Member

@massie massie commented Apr 22, 2014

The new Spark SparkConf object replaces the old configuration system that uses
Java system properties and environment variables. SparkConf is thread-safe and
self-contained. For now, SparkConf is constructed to honor the old system and
environment properties. With time, we should change this behavior.

One effect of this move to SparkConf is to prevent errors around binding a port
for the SparkContext in each test case.

This commit also fixes a race condition in GenotypePredicatesSuites.

The new Spark SparkConf object replaces the old configuration system that uses
Java system properties and environment variables. SparkConf is thread-safe and
self-contained. For now, SparkConf is constructed to honor the old system and
environment properties. With time, we should change this behavior.

One effect of this move to SparkConf is to prevent errors around binding a port
for the SparkContext in each test case.

This commit also fixes a race condition in GenotypePredicatesSuites.
@AmplabJenkins
Copy link

All automated tests passed.
Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/ADAM-prb/297/

fnothaft added a commit that referenced this pull request Apr 23, 2014
Use SparkConf object to configure SparkContext
@fnothaft fnothaft merged commit cf42239 into bigdatagenomics:master Apr 23, 2014
@fnothaft
Copy link
Member

Merged! Thanks @massie! @nealsid, this should fix the issues you reported about @AmplabJenkins earlier today.

@massie massie deleted the sparkconf branch September 2, 2015 19:24
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