Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/3395
---
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 liancheng commented on the pull request:
https://github.com/apache/spark/pull/3395#issuecomment-66915681
LGTM, 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 not have this feature
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/3395#issuecomment-66862821
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3395#issuecomment-66862819
[Test build #24425 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24425/consoleFull)
for PR 3395 at commit
[`ea0e495`](https://gith
Github user wangxiaojing commented on the pull request:
https://github.com/apache/spark/pull/3395#issuecomment-66860102
@liancheng Sorry for the delay in my styling issues, and thanks for your
help.
---
If your project is set up for it, you can reply to this email and have your
rep
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3395#issuecomment-66859945
[Test build #24425 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24425/consoleFull)
for PR 3395 at commit
[`ea0e495`](https://githu
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/3395#issuecomment-66734604
LGTM except for styling issues, 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
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/3395#discussion_r21728943
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/JoinSuite.scala ---
@@ -80,8 +80,11 @@ class JoinSuite extends QueryTest with
BeforeAndAfterEach {
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/3395#discussion_r21728940
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/JoinSuite.scala ---
@@ -80,8 +80,11 @@ class JoinSuite extends QueryTest with
BeforeAndAfterEach {
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/3395#discussion_r21726677
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/JoinSuite.scala ---
@@ -80,8 +80,11 @@ class JoinSuite extends QueryTest with
BeforeAndAfterEach {
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3395#issuecomment-65171371
[Test build #24009 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24009/consoleFull)
for PR 3395 at commit
[`53c3952`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/3395#issuecomment-65171373
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3395#issuecomment-65165713
[Test build #24009 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24009/consoleFull)
for PR 3395 at commit
[`53c3952`](https://githu
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/3395#issuecomment-65165293
ok to test
---
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
ena
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/3395#issuecomment-63918766
Can one of the admins verify this patch?
---
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 pro
GitHub user wangxiaojing opened a pull request:
https://github.com/apache/spark/pull/3395
[SPARK-4527][SQl]Add BroadcastNestedLoopJoin operator selection testsuite
In `JoinSuite` add BroadcastNestedLoopJoin operator selection testsuite
You can merge this pull request into a Git repo
16 matches
Mail list logo
