Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12600#issuecomment-213567775
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12600#issuecomment-213567771
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12600#issuecomment-213567556
**[Test build #56715 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56715/consoleFull)**
for PR 12600 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12600#issuecomment-213565318
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12600#issuecomment-213565316
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12600#issuecomment-213565062
**[Test build #56714 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56714/consoleFull)**
for PR 12600 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12600
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/12600#issuecomment-213562402
Merging this into master, thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12600#issuecomment-213562111
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12600#issuecomment-213562108
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12600#issuecomment-213561607
**[Test build #56713 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56713/consoleFull)**
for PR 12600 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12600#issuecomment-213542932
**[Test build #56715 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56715/consoleFull)**
for PR 12600 at commit
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/12600#issuecomment-213539712
Please add comments to `doPrepare` to mention the lock. Otherwise LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12600#issuecomment-213537141
**[Test build #56714 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56714/consoleFull)**
for PR 12600 at commit
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/12600#discussion_r60779639
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlan.scala ---
@@ -187,11 +187,11 @@ abstract class SparkPlan extends
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/12600#discussion_r60779425
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlan.scala ---
@@ -187,11 +187,11 @@ abstract class SparkPlan extends
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12600#issuecomment-213531619
**[Test build #56713 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56713/consoleFull)**
for PR 12600 at commit
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/12600#discussion_r60778555
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlan.scala ---
@@ -187,11 +187,11 @@ abstract class SparkPlan extends
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/12600#issuecomment-213529482
So `SparkPlan` must be thread-safe? Looks `subqueryResults` is not
protected.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12600#issuecomment-213287306
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12600#issuecomment-213287309
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12600#issuecomment-213286426
**[Test build #56648 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56648/consoleFull)**
for PR 12600 at commit
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12600#issuecomment-213257569
Is there a way to create a test case?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12600#issuecomment-213257260
**[Test build #56648 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56648/consoleFull)**
for PR 12600 at commit
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/12600#issuecomment-213257084
cc @zsxwing @rxin
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
GitHub user davies opened a pull request:
https://github.com/apache/spark/pull/12600
[SPARK-14791] [SQL] fix risk condition between broadcast and subquery
## What changes were proposed in this pull request?
SparkPlan.prepare() could be called in different threads
26 matches
Mail list logo