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

Spark: Recommend using singleton Beam. #115

Merged
merged 1 commit into from
Feb 5, 2016

Conversation

gianm
Copy link
Member

@gianm gianm commented Feb 5, 2016

Fixes #114

@gianm gianm force-pushed the spark-singleton branch 2 times, most recently from 8af3a6f to fef638f Compare February 5, 2016 16:52
@gianm gianm changed the title Spark: Recommend using singleton Beam. Spark: Recommend using singleton Beam, use IdentityHashMap to store Tranquilizers. Feb 5, 2016
@gianm gianm force-pushed the spark-singleton branch 3 times, most recently from faf50f6 to 2d5c7ac Compare February 5, 2016 17:01
@gianm gianm changed the title Spark: Recommend using singleton Beam, use IdentityHashMap to store Tranquilizers. Spark: Recommend using singleton Beam. Feb 5, 2016
@fjy
Copy link
Member

fjy commented Feb 5, 2016

👍

@@ -88,27 +86,39 @@ class SparkDruidTest
timekeeper.now = now
}
)

ssc.stop(true, true)
ssc.awaitTermination()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this timeout protected somehow?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh test, nvmnd

drcrallen added a commit that referenced this pull request Feb 5, 2016
Spark: Recommend using singleton Beam.
@drcrallen drcrallen merged commit 697eb2c into druid-io:master Feb 5, 2016
@gianm gianm deleted the spark-singleton branch February 11, 2016 21:35
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.

3 participants