Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8854
---
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 enab
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142411104
Thank you for reviewing this PR! I am merging it to master and branch 1.5.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142406661
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
enable
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142397298
@JoshRosen Can you also take a look?
---
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 n
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142392425
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
enabled
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8854#discussion_r40117487
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeJoin.scala
---
@@ -82,6 +79,28 @@ case class SortMergeJoin(
left
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142209000
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142208999
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142208903
[Test build #42816 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42816/console)
for PR 8854 at commit
[`55d4f9a`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142188914
[Test build #42816 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42816/consoleFull)
for PR 8854 at commit
[`55d4f9a`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142187995
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142188011
Merged build started.
---
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 yhuai commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142187898
test this please
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142187580
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142187556
[Test build #42809 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42809/console)
for PR 8854 at commit
[`55d4f9a`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142187581
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142175643
[Test build #42809 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42809/consoleFull)
for PR 8854 at commit
[`55d4f9a`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142174513
Merged build started.
---
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/8854#issuecomment-142174468
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142167892
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 project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142167893
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142167866
[Test build #42798 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42798/console)
for PR 8854 at commit
[`5406680`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142156700
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142156699
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142156677
[Test build #42790 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42790/console)
for PR 8854 at commit
[`3a948d2`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142155476
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142155475
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142155009
[Test build #42798 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42798/consoleFull)
for PR 8854 at commit
[`5406680`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142154047
Merged build started.
---
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/8854#issuecomment-142154007
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142152374
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142152403
Merged build started.
---
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 yhuai commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142152351
jenkins test this please
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142144037
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 project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142144038
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142144007
[Test build #42789 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42789/console)
for PR 8854 at commit
[`3c8f8cb`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142142054
[Test build #42790 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42790/consoleFull)
for PR 8854 at commit
[`3a948d2`](https://gith
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/8854#discussion_r40041797
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeJoin.scala
---
@@ -82,6 +79,28 @@ case class SortMergeJoin(
l
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142140483
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142140530
Merged build started.
---
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 yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8854#discussion_r40041316
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -1781,4 +1781,32 @@ class SQLQuerySuite extends QueryTest with
SharedSQLConte
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/8854#discussion_r40041167
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -1781,4 +1781,32 @@ class SQLQuerySuite extends QueryTest with
SharedSQLCo
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142137166
[Test build #42789 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42789/consoleFull)
for PR 8854 at commit
[`3c8f8cb`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142136861
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142136875
Merged build started.
---
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/8854#issuecomment-142135176
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 project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142135177
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142135175
[Test build #42783 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42783/console)
for PR 8854 at commit
[`5bff8dc`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142124800
[Test build #42783 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42783/consoleFull)
for PR 8854 at commit
[`5bff8dc`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8854#issuecomment-142124032
Merged build started.
---
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/8854#issuecomment-142123998
Merged build triggered.
---
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 h
GitHub user yhuai opened a pull request:
https://github.com/apache/spark/pull/8854
[SPARK-10737] [SQL] When using UnsafeRows, SortMergeJoin may return wrong
results
https://issues.apache.org/jira/browse/SPARK-10737
You can merge this pull request into a Git repository by running:
52 matches
Mail list logo