Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11831#issuecomment-198549576
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 hhbyyh commented on the pull request:
https://github.com/apache/spark/pull/11536#issuecomment-197784888
Thanks. Let me know if I can help in any way.
---
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/11816#issuecomment-198245670
**[Test build #53522 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53522/consoleFull)**
for PR 11816 at commit
Github user sameeragarwal closed the pull request at:
https://github.com/apache/spark/pull/11808
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11769#issuecomment-197880442
**[Test build #53429 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53429/consoleFull)**
for PR 11769 at commit
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/11790#issuecomment-198066796
@hhbyyh Thanks for reviewing it! I'll merge it with master
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11762#discussion_r56330360
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/parser/CatalystQlSuite.scala
---
@@ -171,6 +171,7 @@ class CatalystQlSuite extends
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/11105#discussion_r56414687
--- Diff:
core/src/test/scala/org/apache/spark/ConsistentAccumulatorsSuite.scala ---
@@ -0,0 +1,284 @@
+/*
+ * Licensed to the Apache Software
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/11105#discussion_r56407098
--- Diff: core/src/main/scala/org/apache/spark/util/collection/Utils.scala
---
@@ -36,4 +36,21 @@ private[spark] object Utils {
}
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10231#issuecomment-198503860
**[Test build #53553 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53553/consoleFull)**
for PR 10231 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11273#issuecomment-197627680
**[Test build #53377 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53377/consoleFull)**
for PR 11273 at commit
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/11674#issuecomment-197977531
LGTM, merging into 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 rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11815#discussion_r56618261
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/planning/patterns.scala
---
@@ -202,3 +203,14 @@ object Unions {
}
}
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11782#issuecomment-198274700
**[Test build #53524 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53524/consoleFull)**
for PR 11782 at commit
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/11768#discussion_r56408146
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/SQLBuilder.scala ---
@@ -368,6 +398,14 @@ class SQLBuilder(logicalPlan: LogicalPlan, sqlContext:
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11768#issuecomment-197541174
**[Test build #53339 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53339/consoleFull)**
for PR 11768 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11768#issuecomment-197510253
**[Test build #53339 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53339/consoleFull)**
for PR 11768 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11813#issuecomment-198218704
**[Test build #53505 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53505/consoleFull)**
for PR 11813 at commit
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/11536#discussion_r56709488
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/CountVectorizer.scala ---
@@ -233,7 +254,13 @@ class CountVectorizerModel(override val uid:
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11620#issuecomment-197558360
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/11747#issuecomment-197585823
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11709#issuecomment-197589892
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11794#issuecomment-198079470
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11828#issuecomment-198585995
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/11764#issuecomment-197697735
**[Test build #53391 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53391/consoleFull)**
for PR 11764 at commit
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/11696#discussion_r56497874
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/SQLBuilder.scala ---
@@ -221,6 +224,42 @@ class SQLBuilder(logicalPlan: LogicalPlan,
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/11673#discussion_r56599961
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1450,7 +1450,9 @@ object CleanupAliases extends
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11701#issuecomment-197480713
**[Test build #5 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/5/consoleFull)**
for PR 11701 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11722#issuecomment-197841373
**[Test build #53423 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53423/consoleFull)**
for PR 11722 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11799#issuecomment-198136453
**[Test build #53476 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53476/consoleFull)**
for PR 11799 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11273#issuecomment-197607628
**[Test build #53377 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53377/consoleFull)**
for PR 11273 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11318#issuecomment-197619726
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11735#issuecomment-197286860
Jenkins retest 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 marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/11645#discussion_r56374099
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/HDFSBackedStateStoreProvider.scala
---
@@ -0,0 +1,465 @@
+/*
+ *
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11318#issuecomment-197579448
**[Test build #53360 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53360/consoleFull)**
for PR 11318 at commit
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11701#discussion_r56409783
--- Diff: python/run-tests.py ---
@@ -54,10 +54,27 @@ def print_red(text):
LOGGER = logging.getLogger()
-def
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/11645#discussion_r56376021
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/streaming/state/StateStoreSuite.scala
---
@@ -0,0 +1,471 @@
+/*
+ * Licensed to
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9229#issuecomment-198094792
**[Test build #53459 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53459/consoleFull)**
for PR 9229 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11821#issuecomment-198318838
**[Test build #53533 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53533/consoleFull)**
for PR 11821 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11821#issuecomment-198317863
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/11645#discussion_r56373801
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/StateStore.scala
---
@@ -0,0 +1,200 @@
+/*
+ * Licensed to the
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11801#issuecomment-198139336
**[Test build #53481 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53481/consoleFull)**
for PR 11801 at commit
Github user zhengruifeng commented on the pull request:
https://github.com/apache/spark/pull/11821#issuecomment-198313360
cc @MLnick
---
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/11655#issuecomment-197277639
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/10839#issuecomment-197557655
@Fokko That PR was merged. Could you merge the current master and update
your implementation? Note that when you concat the vectors, it is useful to
check the sparsity
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11749#issuecomment-197595608
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 marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/11645#discussion_r56373579
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/StateStore.scala
---
@@ -0,0 +1,200 @@
+/*
+ * Licensed to the
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11776#issuecomment-197757480
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/11410#issuecomment-198407456
**[Test build #53541 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53541/consoleFull)**
for PR 11410 at commit
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/11828#discussion_r56744115
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/ColumnPruningSuite.scala
---
@@ -35,7 +35,8 @@ class ColumnPruningSuite
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11557#issuecomment-198167945
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 HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/11806#issuecomment-198326532
@srowen @steveloughram Thank you so much. Could anybody please decide if I
should go head or not? For me it's a bit confusing. I will follow the decision.
---
If
Github user jodersky commented on the pull request:
https://github.com/apache/spark/pull/11708#issuecomment-197596487
@rxin, your patch should work on top of this commit now
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11815#issuecomment-198487974
cc @cloud-fan for 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/11827#issuecomment-198560082
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/11447#issuecomment-197883525
**[Test build #53431 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53431/consoleFull)**
for PR 11447 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11809#issuecomment-198209846
**[Test build #53495 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53495/consoleFull)**
for PR 11809 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11806#issuecomment-198142708
**[Test build #53484 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53484/consoleFull)**
for PR 11806 at commit
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11775#issuecomment-198196289
cc @liancheng
---
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 davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/11792#discussion_r56590282
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/basicOperators.scala ---
@@ -110,23 +114,34 @@ case class Filter(condition: Expression,
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11708#issuecomment-197595666
**[Test build #53356 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53356/consoleFull)**
for PR 11708 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11684#issuecomment-197443339
**[Test build #53327 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53327/consoleFull)**
for PR 11684 at commit
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11784#discussion_r56517612
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -508,12 +508,10 @@ class LogisticRegressionModel
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/11764#issuecomment-197408221
We made the `Logging` class private and broke user code. To minimize the
code change, users can create `org.apache.spark.Logging` class themselves.
However, we have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11791#issuecomment-198087068
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11775#discussion_r56475177
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -205,7 +205,17 @@ case class DataSource(
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11301#issuecomment-198237085
**[Test build #53517 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53517/consoleFull)**
for PR 11301 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11840#issuecomment-198651067
**[Test build #53610 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53610/consoleFull)**
for PR 11840 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8112#issuecomment-197808444
**[Test build #53406 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53406/consoleFull)**
for PR 8112 at commit
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/11765#issuecomment-198534808
@gatorsmile Should we fix this in InferFiltersFromConstraints itself?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user hbhanawat commented on the pull request:
https://github.com/apache/spark/pull/11723#issuecomment-197804928
@rxin ok, I get it. I would make ExternalClusterManager as private[spark]
and mark it as developer API. I hope that should suffice.
---
If your project is set up
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11645#issuecomment-197671953
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11701#issuecomment-198010175
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/11796#issuecomment-198138465
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 gatorsmile opened a pull request:
https://github.com/apache/spark/pull/11765
Constraint filter
What changes were proposed in this pull request?
In the query, Optimizer generates many duplicate constraints in Filter
constraints by applying the rule
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11776#issuecomment-197757077
**[Test build #53405 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53405/consoleFull)**
for PR 11776 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11735#issuecomment-197277276
**[Test build #53311 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53311/consoleFull)**
for PR 11735 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11764#issuecomment-197681104
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/11720#issuecomment-197905769
**[Test build #53425 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53425/consoleFull)**
for PR 11720 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11828#issuecomment-198534281
**[Test build #53560 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53560/consoleFull)**
for PR 11828 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11769#issuecomment-197861183
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/11756#issuecomment-198214357
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/9229#issuecomment-198095370
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/11773#issuecomment-197630350
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/11765#issuecomment-197448448
**[Test build #53328 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53328/consoleFull)**
for PR 11765 at commit
Github user dongjoon-hyun commented on the pull request:
https://github.com/apache/spark/pull/11773#issuecomment-197954539
Thank you, @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
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11682#discussion_r56459726
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -410,6 +401,24 @@ object ColumnPruning extends
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11823#issuecomment-198326620
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/11636#issuecomment-198109532
**[Test build #53470 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53470/consoleFull)**
for PR 11636 at commit
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/11108#discussion_r56434528
--- Diff: examples/src/main/python/mllib/hypothesis_testing_example.py ---
@@ -0,0 +1,63 @@
+#
+# Licensed to the Apache Software Foundation (ASF)
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/9884#discussion_r56428323
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/feature/RFormulaSuite.scala ---
@@ -214,4 +215,41 @@ class RFormulaSuite extends SparkFunSuite with
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11301#issuecomment-19821
**[Test build #53521 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53521/consoleFull)**
for PR 11301 at commit
Github user wangyang1992 closed the pull request at:
https://github.com/apache/spark/pull/11762
---
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
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11788#issuecomment-198198599
I was thinking an option to prefer sort merge join. My main concern is that
it is easier to oom when there are skew in hash joins.
---
If your project is set up for it,
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/11788#discussion_r56616530
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/LeftSemiJoinHash.scala
---
@@ -1,58 +0,0 @@
-/*
- * Licensed to the Apache
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-198009919
**[Test build #53445 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53445/consoleFull)**
for PR 11748 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11812#issuecomment-198194974
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 thunterdb commented on a diff in the pull request:
https://github.com/apache/spark/pull/11802#discussion_r56595285
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/StreamingContext.scala ---
@@ -132,7 +132,7 @@ class StreamingContext private[streaming] (
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/11750#issuecomment-197627234
OK. Let me merge this. Let's address comments in the follow-up pr.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11301#issuecomment-197460999
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
1001 - 1100 of 1811 matches
Mail list logo
