Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12668
---
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/12668#issuecomment-214863380
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 hvanhovell commented on the pull request:
https://github.com/apache/spark/pull/12668#issuecomment-214862732
LGTM
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12668#issuecomment-214862249
**[Test build #2886 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2886/consoleFull)**
for PR 12668 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12668#issuecomment-214833920
**[Test build #2886 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2886/consoleFull)**
for PR 12668 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12668#issuecomment-214832015
Merged build finished. Test FAILed.
---
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/12668#issuecomment-214831825
**[Test build #57006 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57006/consoleFull)**
for PR 12668 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12668#issuecomment-214832021
Test FAILed.
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/12668#issuecomment-214807707
**[Test build #57006 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57006/consoleFull)**
for PR 12668 at commit
Github user hvanhovell commented on the pull request:
https://github.com/apache/spark/pull/12668#issuecomment-214760036
@davies I left a few comments. Only one of them is more serious.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/12668#discussion_r61089605
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -131,76 +116,80 @@ private[sql] abstract class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/12668#discussion_r61089122
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -107,16 +80,28 @@ private[sql] abstract class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/12668#discussion_r61088744
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -107,16 +80,28 @@ private[sql] abstract class
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12668#issuecomment-214661732
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/12668#issuecomment-214661730
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/12668#issuecomment-214661502
**[Test build #56977 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56977/consoleFull)**
for PR 12668 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12668#issuecomment-214655653
Test FAILed.
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/12668#issuecomment-214655387
**[Test build #56976 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56976/consoleFull)**
for PR 12668 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12668#issuecomment-214655648
Merged build finished. Test FAILed.
---
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/12668#issuecomment-214629764
**[Test build #56977 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56977/consoleFull)**
for PR 12668 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12668#issuecomment-214627943
**[Test build #56976 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56976/consoleFull)**
for PR 12668 at commit
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12668#issuecomment-214610321
@hvanhovell want to review?
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12668#issuecomment-214574085
Test FAILed.
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/12668#issuecomment-214574083
Merged build finished. Test FAILed.
---
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/12668#issuecomment-214573986
**[Test build #56941 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56941/consoleFull)**
for PR 12668 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12668#issuecomment-214561234
**[Test build #56941 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56941/consoleFull)**
for PR 12668 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12668#issuecomment-214536972
Test FAILed.
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/12668#issuecomment-214536971
Merged build finished. Test FAILed.
---
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/12668#issuecomment-214536711
**[Test build #56923 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56923/consoleFull)**
for PR 12668 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12668#issuecomment-214518004
**[Test build #56923 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56923/consoleFull)**
for PR 12668 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12668#issuecomment-214515327
**[Test build #56921 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56921/consoleFull)**
for PR 12668 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12668#issuecomment-214515338
Test FAILed.
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/12668#issuecomment-214515335
Merged build finished. Test FAILed.
---
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/12668#issuecomment-214514873
**[Test build #56921 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56921/consoleFull)**
for PR 12668 at commit
GitHub user davies opened a pull request:
https://github.com/apache/spark/pull/12668
[SPARK-14853] [SQL] Support LeftSemi/LeftAnti in SortMergeJoinExec
## What changes were proposed in this pull request?
This PR update SortMergeJoinExec to support LeftSemi/LeftAnti, so it
35 matches
Mail list logo