-
Notifications
You must be signed in to change notification settings - Fork 28.2k
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-7044][SQL] Fix the deadlock in ScriptTransform(for Spark 1.3) #5671
Conversation
Test build #30891 has started for PR 5671 at commit
|
Test build #30891 has finished for PR 5671 at commit
|
Test PASSed. |
cc @rxin |
Thanks. I've merged this. |
Can you close the PR? Since it was not merged into master, github won't close this automatically. |
Thank you @rxin, closing since it's merged. |
@chenghao-intel The exact deadlock happens for stderr too, is there a quick fix for that? Thanks! |
Thank you @jfhu for reporting this, I will do the updating. And it will be appreciated if you can paste the reproduce steps. |
@chenghao-intel Thanks! re-pro steps: (actually copied and adapted from your patch :)
|
[Related PR SPARK-7044] (#5671) Author: zhichao.li <zhichao.li@intel.com> Closes #6404 from zhichao-li/transform and squashes the following commits: 8418c97 [zhichao.li] add comments and remove useless failAfter logic d9677e1 [zhichao.li] redirect the error desitination to be the same as the current process
[Related PR SPARK-7044] (apache#5671) Author: zhichao.li <zhichao.li@intel.com> Closes apache#6404 from zhichao-li/transform and squashes the following commits: 8418c97 [zhichao.li] add comments and remove useless failAfter logic d9677e1 [zhichao.li] redirect the error desitination to be the same as the current process
No description provided.