-
Notifications
You must be signed in to change notification settings - Fork 29k
Mend StreamingContext stop() followed by awaitTermination(). #1001
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
Conversation
|
Can one of the admins verify this patch? |
|
This patch is my original work, and I accept to have it licensed under Apache v2. |
|
Jenkins, test this please. LGTM - good catch on this, thanks for the fix! |
|
Hey @lallea mind quickly creating a JIRA for this? We try to do this with any user-facing bug so if someone runs into it they can quickly figure out which version it is fixed in. |
|
Merged build triggered. |
|
Merged build started. |
|
Merged build finished. All automated tests passed. |
|
All automated tests passed. |
Author: Lars Albertsson <[email protected]> Closes #1001 from lallea/contextwaiter_stopped and squashes the following commits: 93cd314 [Lars Albertsson] Mend StreamingContext stop() followed by awaitTermination(). (cherry picked from commit 4d5c12a) Signed-off-by: Patrick Wendell <[email protected]>
|
You beat me to Jira. Thanks for the merge. |
Author: Lars Albertsson <[email protected]> Closes apache#1001 from lallea/contextwaiter_stopped and squashes the following commits: 93cd314 [Lars Albertsson] Mend StreamingContext stop() followed by awaitTermination().
Author: Lars Albertsson <[email protected]> Closes apache#1001 from lallea/contextwaiter_stopped and squashes the following commits: 93cd314 [Lars Albertsson] Mend StreamingContext stop() followed by awaitTermination().
No description provided.