Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11288
---
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 rxin commented on the pull request:
https://github.com/apache/spark/pull/11288#issuecomment-186895449
Thanks - merging in master.
---
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 dongjoon-hyun commented on the pull request:
https://github.com/apache/spark/pull/11289#issuecomment-186894117
Oh, thank you, @srowen .
---
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
Github user xguo27 commented on the pull request:
https://github.com/apache/spark/pull/11291#issuecomment-186893918
@hvanhovell In hashSemiJoin() function, when condition is empty, the
boundCondition always evaluates to true here:
Github user hvanhovell commented on the pull request:
https://github.com/apache/spark/pull/11288#issuecomment-186893722
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/11291#issuecomment-186892930
**[Test build #2558 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2558/consoleFull)**
for PR 11291 at commit
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53571306
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/SparkQlSuite.scala ---
@@ -0,0 +1,652 @@
+/*
+ * Licensed to the Apache Software
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53571294
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/commands.scala
---
@@ -0,0 +1,533 @@
+/*
--- End diff --
+1
---
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53571196
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53571180
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user hvanhovell commented on the pull request:
https://github.com/apache/spark/pull/11291#issuecomment-186888525
I don't think I have enough karma to trigger a build... @rxin could you
trigger a build?
---
If your project is set up for it, you can reply to this email and have
Github user hvanhovell commented on the pull request:
https://github.com/apache/spark/pull/11291#issuecomment-186885302
@xguo27 Could you also take a look at the `hashSemiJoin(...)` function and
see how this treats an empty condition?
---
If your project is set up for it, you can
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11291#discussion_r53571028
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/BroadcastLeftSemiJoinHash.scala
---
@@ -45,18 +45,15 @@ case class
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11292#issuecomment-186882750
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/11292#issuecomment-186882748
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/11292#issuecomment-186882701
**[Test build #51642 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51642/consoleFull)**
for PR 11292 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11292#issuecomment-186880803
**[Test build #51642 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51642/consoleFull)**
for PR 11292 at commit
Github user xguo27 commented on the pull request:
https://github.com/apache/spark/pull/11291#issuecomment-186878372
@hvanhovell I see, sorry for my lack of patience. : )
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user hvanhovell commented on the pull request:
https://github.com/apache/spark/pull/11291#issuecomment-186876984
It usually takes about 15 min before a test is triggered
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user xguo27 commented on the pull request:
https://github.com/apache/spark/pull/11291#issuecomment-186876120
Looks like the command did not trigger a 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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11133#issuecomment-186875930
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/11133#issuecomment-186875929
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/11133#issuecomment-186875823
**[Test build #51639 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51639/consoleFull)**
for PR 11133 at commit
GitHub user srowen opened a pull request:
https://github.com/apache/spark/pull/11292
[SPARK-13423] [WIP] [CORE] [SQL] [STREAMING] Static analysis fixes for 2.x
## What changes were proposed in this pull request?
Make some cross-cutting code improvements according to static
Github user hvanhovell commented on the pull request:
https://github.com/apache/spark/pull/11291#issuecomment-186875493
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11291#issuecomment-186872985
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
Github user xguo27 commented on the pull request:
https://github.com/apache/spark/pull/11291#issuecomment-186872660
@hvanhovell Could you please advise whether this is the right fix? All Left
Semi related tests passed, but I'm not sure what other impact there might be to
remove
GitHub user xguo27 opened a pull request:
https://github.com/apache/spark/pull/11291
[SPARK-13422][SQL] Use HashedRelation instead of HashSet in Left Semi Joins
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/xguo27/spark
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53570110
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53570023
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53570024
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53570001
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53569784
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7#issuecomment-186862480
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 hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53569740
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53569754
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7#issuecomment-186862481
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53569704
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7#issuecomment-186862424
**[Test build #51638 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51638/consoleFull)**
for PR 7 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11132#issuecomment-186862379
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/11132#issuecomment-186862377
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/11132#issuecomment-186862336
**[Test build #51641 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51641/consoleFull)**
for PR 11132 at commit
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53569677
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10058#issuecomment-186862236
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/10058#issuecomment-186862237
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/10058#issuecomment-186862167
**[Test build #51636 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51636/consoleFull)**
for PR 10058 at commit
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53569610
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11290#issuecomment-186860314
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/11290#issuecomment-186860315
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/11290#issuecomment-186860282
**[Test build #51640 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51640/consoleFull)**
for PR 11290 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11132#issuecomment-186859997
**[Test build #51641 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51641/consoleFull)**
for PR 11132 at commit
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53569457
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53569392
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11083#issuecomment-186858134
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/11083#issuecomment-186858132
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/11083#issuecomment-186857661
**[Test build #51637 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51637/consoleFull)**
for PR 11083 at commit
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53569287
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11286#discussion_r53569281
--- Diff:
streaming/src/test/scala/org/apache/spark/streaming/CheckpointSuite.scala ---
@@ -133,6 +133,17 @@ trait DStreamCheckpointTester { self:
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11083#issuecomment-186856561
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/11083#issuecomment-186856563
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53569257
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11083#issuecomment-186856355
**[Test build #51635 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51635/consoleFull)**
for PR 11083 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11290#issuecomment-186856257
**[Test build #51640 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51640/consoleFull)**
for PR 11290 at commit
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11287#issuecomment-186856227
This looks conceptually fine, though the test failed. I am not sure why. I
stared at this for a while and don't see what would cause it to not be limited
to 8 threads
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53569223
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53569202
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53569174
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53569109
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53569098
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11133#issuecomment-186855426
**[Test build #51639 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51639/consoleFull)**
for PR 11133 at commit
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53569083
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user devaraj-kavali commented on the pull request:
https://github.com/apache/spark/pull/11053#issuecomment-186855380
Thanks @yinxusen for the good suggestion, I have addressed it.
> ModelSelectionViaTrainValidationSplitExample and
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11290#issuecomment-186855315
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
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11237#issuecomment-186855236
OK by me though ideally @mengxr would merge. I'll do it tomorrow if not
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11201#discussion_r53568953
--- Diff: docs/streaming-programming-guide.md ---
@@ -1787,11 +1788,11 @@ reliable file system (e.g., HDFS, S3, etc.) to
which the checkpoint information
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11133#issuecomment-186854471
@devaraj-kavali Let's try one more time. I'm not seeing that failure in
other builds. Although the exception's ultimate cause isn't so clear, it could
still be due to
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11139
---
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 srowen commented on the pull request:
https://github.com/apache/spark/pull/11133#issuecomment-186854431
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
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11139#issuecomment-186854347
Merged to master
---
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 hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53568615
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53568504
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/parsers.scala
---
@@ -0,0 +1,420 @@
+/*
+ * Licensed to the Apache
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/10527#discussion_r53568446
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -1931,6 +1931,42 @@ object functions extends LegacyFunctions {
new
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53568236
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/commands.scala
---
@@ -0,0 +1,533 @@
+/*
+ * Licensed to the Apache
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53568197
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands/commands.scala
---
@@ -0,0 +1,533 @@
+/*
--- End diff --
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53568183
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkQl.scala ---
@@ -62,6 +87,57 @@ private[sql] class SparkQl(conf: ParserConf =
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7#issuecomment-186843839
**[Test build #51638 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51638/consoleFull)**
for PR 7 at commit
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53568012
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkQl.scala ---
@@ -16,18 +16,43 @@
*/
package
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11289#issuecomment-186842357
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/11289#issuecomment-186842358
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53567861
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/PlanParser.scala ---
@@ -0,0 +1,53 @@
+/*
+ * Licensed to the Apache Software
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11289#issuecomment-186842274
**[Test build #51629 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51629/consoleFull)**
for PR 11289 at commit
Github user lianhuiwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/10058#discussion_r53567785
--- Diff: core/src/main/scala/org/apache/spark/ui/exec/ExecutorsPage.scala
---
@@ -159,10 +181,14 @@ private[ui] class ExecutorsPage(
}
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6042#issuecomment-186841902
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/10058#issuecomment-186841946
**[Test build #51636 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51636/consoleFull)**
for PR 10058 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6042#issuecomment-186841905
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53567755
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystQl.scala ---
@@ -90,7 +93,7 @@ private[sql] class CatalystQl(val conf:
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6042#issuecomment-186841743
**[Test build #51632 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51632/consoleFull)**
for PR 6042 at commit
Github user insidedctm commented on the pull request:
https://github.com/apache/spark/pull/11290#issuecomment-186838624
@srowen good points, I've updated and pushed changes in line with your
comments
---
If your project is set up for it, you can reply to this email and have your
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11083#issuecomment-186838836
**[Test build #51637 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51637/consoleFull)**
for PR 11083 at commit
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11048#discussion_r53567674
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystQl.scala ---
@@ -30,17 +30,20 @@ import org.apache.spark.sql.types._
201 - 300 of 429 matches
Mail list logo
