Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/11836#discussion_r56745379
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveCatalog.scala ---
@@ -182,13 +189,15 @@ private[spark] class HiveCatalog(client: HiveClien
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/11836#discussion_r56745391
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala ---
@@ -81,15 +83,31 @@ class HiveContext private[hive](
sc: SparkC
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/11841#issuecomment-19864
LGTM except for one MiMA check question.
---
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 proje
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11840#issuecomment-198648880
**[Test build #53608 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53608/consoleFull)**
for PR 11840 at commit
[`0cc3f4f`](https://gi
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/11841#discussion_r56745324
--- Diff: project/MimaExcludes.scala ---
@@ -315,6 +315,7 @@ object MimaExcludes {
ProblemFilters.exclude[MissingClassProblem]("org.apache.spa
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11831#issuecomment-198648881
**[Test build #53609 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53609/consoleFull)**
for PR 11831 at commit
[`70b41c8`](https://gi
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11840#discussion_r56745275
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -826,6 +827,17 @@ object CombineFilters extends Rule
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11840#discussion_r56745280
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -826,6 +827,17 @@ object CombineFilters extends Rule
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11841#issuecomment-198648728
**[Test build #53607 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53607/consoleFull)**
for PR 11841 at commit
[`3620da9`](https://gi
Github user dongjoon-hyun commented on the pull request:
https://github.com/apache/spark/pull/11838#issuecomment-198648575
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 have th
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/11815#issuecomment-198634566
LGTM except 2 minor comments
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11841#issuecomment-198648512
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11841#issuecomment-198648504
**[Test build #53606 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53606/consoleFull)**
for PR 11841 at commit
[`24eaf42`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11841#issuecomment-198648514
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user yinxusen commented on the pull request:
https://github.com/apache/spark/pull/11835#issuecomment-198634432
@jkbradley
MiMa tests failed for changing to the `StageArrayParam`. But I think we
need a new Param like `ArrayParam[T]` with the Java compatible `w` function
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11791#issuecomment-198648221
Still WIP?
---
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
enab
Github user viirya closed the pull request at:
https://github.com/apache/spark/pull/11817
---
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 ena
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/11817#issuecomment-198648163
Rethink this issue, I think the issue described in the JIRA should not
related to pushdown of limit. Because the latest CollectLimit only takes few
rows (here is only 1
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11840#discussion_r56745107
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -826,6 +827,17 @@ object CombineFilters extends Rule[Logic
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11834#issuecomment-198634733
Did any code change?
---
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 fea
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11841#issuecomment-198648078
**[Test build #53606 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53606/consoleFull)**
for PR 11841 at commit
[`24eaf42`](https://gi
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11840#discussion_r56745083
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -826,6 +827,17 @@ object CombineFilters extends Rule[Logic
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11839#issuecomment-198635870
**[Test build #53603 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53603/consoleFull)**
for PR 11839 at commit
[`cbe436f`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11722#issuecomment-198640266
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 projec
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11841#issuecomment-198647907
cc @liancheng and @sameeragarwal
---
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 rxin opened a pull request:
https://github.com/apache/spark/pull/11841
[SPARK-13897][SQL] RelationalGroupedDataset and KeyValueGroupedDataset
## What changes were proposed in this pull request?
Previously, Dataset.groupBy returns a GroupedData, and Dataset.groupByKey
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11828#issuecomment-198647760
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11828#issuecomment-198647761
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11828#issuecomment-198647714
**[Test build #53600 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53600/consoleFull)**
for PR 11828 at commit
[`b1118e5`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11840#issuecomment-198646675
**[Test build #53605 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53605/consoleFull)**
for PR 11840 at commit
[`96cd8ce`](https://gi
Github user sameeragarwal commented on the pull request:
https://github.com/apache/spark/pull/11834#issuecomment-198637517
No (we made all the changes in https://github.com/apache/spark/pull/11799)
---
If your project is set up for it, you can reply to this email and have your
reply a
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11807#issuecomment-198638912
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 projec
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/11840
[Spark-14019] [SQL] Remove noop SortOrder in Sort
What changes were proposed in this pull request?
This PR is to add a new Optimizer rule for pruning Sort if its SortOrder is
no-op
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11807#issuecomment-198638752
**[Test build #53602 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53602/consoleFull)**
for PR 11807 at commit
[`d31bb3f`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11784#issuecomment-197916455
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/11794#discussion_r56690412
--- Diff:
core/src/main/java/org/apache/spark/shuffle/sort/ShuffleExternalSorter.java ---
@@ -320,7 +320,15 @@ private void growPointerArrayIfNecessary() thro
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/11770#issuecomment-197572471
BTW, if you're going to consider changing `SparkEnv` then I'd remove the
deprecated methods from back when it used to be a thread-local.
---
If your project is set u
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11621#issuecomment-198019144
**[Test build #53449 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53449/consoleFull)**
for PR 11621 at commit
[`d7e17ab`](https://gi
Github user yy2016 commented on the pull request:
https://github.com/apache/spark/pull/11787#issuecomment-198053603
I wonder why OneRowRelation isn't covered by the following import ?
```
import org.apache.spark.sql.catalyst.plans.logical._
```
---
If your project is set u
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11817#issuecomment-198370581
**[Test build #53539 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53539/consoleFull)**
for PR 11817 at commit
[`2d983c9`](https://gi
Github user sethah commented on the pull request:
https://github.com/apache/spark/pull/9008#issuecomment-198428763
cc @MLnick thoughts on the above comments?
---
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 felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/11486#discussion_r56623549
--- Diff: R/pkg/R/mllib.R ---
@@ -71,14 +71,23 @@ setMethod("glm", signature(formula = "formula", family
= "ANY", data = "DataFram
#' @rdname predic
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11750#issuecomment-197596671
**[Test build #2646 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2646/consoleFull)**
for PR 11750 at commit
[`ad43a5f`](https://g
Github user tomwhite commented on the pull request:
https://github.com/apache/spark/pull/11806#issuecomment-198358199
I agree that BLOCK is always to be preferred over RECORD, so leave it at
BLOCK. RECORD is the default in Hadoop 1 and 2 (for backwards compatibility
reasons), but that
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11297#issuecomment-197749675
**[Test build #53397 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53397/consoleFull)**
for PR 11297 at commit
[`29a4f59`](https://g
Github user shaneknapp commented on the pull request:
https://github.com/apache/spark/pull/11652#issuecomment-197597252
i'm currently installing the latest lintr on all of our jenkins workers.
this should finish in ~15 mins
---
If your project is set up for it, you can reply to this
GitHub user antonoal opened a pull request:
https://github.com/apache/spark/pull/11777
Added transitive closure transformation to Catalyst
## What changes were proposed in this pull request?
A relatively simple transformation is missing from Catalyst's arsenal -
generation of tr
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11735#issuecomment-197351217
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-197635547
cc @yhuai
---
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/11737#issuecomment-197702816
**[Test build #53394 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53394/consoleFull)**
for PR 11737 at commit
[`59cae95`](https://gi
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/11750#discussion_r56425133
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala
---
@@ -211,8 +214,7 @@ case class CatalogTablePartition(
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11805#issuecomment-198142117
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/11663#discussion_r56548734
--- Diff: python/pyspark/ml/param/_shared_params_code_gen.py ---
@@ -105,64 +104,71 @@ def get$Name(self):
if __name__ == "__main__":
print(heade
Github user sethah commented on the pull request:
https://github.com/apache/spark/pull/10231#issuecomment-197597688
I can set something up. Do you have a specific dataset size in mind or even
a specific dataset?
---
If your project is set up for it, you can reply to this email and ha
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11709#issuecomment-198553825
**[Test build #53558 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53558/consoleFull)**
for PR 11709 at commit
[`c7ca5fe`](https://g
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/11645#discussion_r56375262
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/StateStoreRDD.scala
---
@@ -0,0 +1,60 @@
+/*
+ * Licensed to the A
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11242#issuecomment-198572980
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/11636#discussion_r56455795
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/BoundAttribute.scala
---
@@ -60,17 +60,28 @@ case class BoundReference(ordina
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/11663#discussion_r56547567
--- Diff: python/pyspark/ml/param/__init__.py ---
@@ -65,6 +72,106 @@ def __eq__(self, other):
return False
+class TypeConvert
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11603#issuecomment-197836044
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user jeanlyn closed the pull request at:
https://github.com/apache/spark/pull/11679
---
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 en
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11824#issuecomment-198463972
**[Test build #53545 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53545/consoleFull)**
for PR 11824 at commit
[`ebf35a3`](https://gi
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/11701#issuecomment-198002898
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11792#issuecomment-198090947
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/11756#discussion_r56325150
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -963,6 +963,28 @@ class JsonSuite extends QueryTe
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/11683#issuecomment-197552039
Hm, if you want to do it under a new JIRA that's OK too. I'll create one
now.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/11745#discussion_r56705349
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -306,14 +311,28 @@ object SetOperationPushDown extends R
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11788#issuecomment-198009917
**[Test build #53444 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53444/consoleFull)**
for PR 11788 at commit
[`2122986`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11755#issuecomment-197620388
**[Test build #53365 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53365/consoleFull)**
for PR 11755 at commit
[`45b0c0b`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11803#issuecomment-198153139
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11791#issuecomment-198185758
**[Test build #53496 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53496/consoleFull)**
for PR 11791 at commit
[`5489748`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11793#issuecomment-198081036
**[Test build #53464 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53464/consoleFull)**
for PR 11793 at commit
[`48061de`](https://g
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11759
---
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 ena
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11799
---
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 ena
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/11794#issuecomment-198517715
Merging this into master and 1.6, 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 tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/11645#discussion_r56387171
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/streaming/state/StateStoreRDDSuite.scala
---
@@ -0,0 +1,147 @@
+/*
+ * Licensed to the
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11707#issuecomment-197553313
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11755#issuecomment-197561059
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 projec
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11815#discussion_r56618306
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -618,9 +622,33 @@ class Analyzer(
* clause. This ru
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/11301#discussion_r56354960
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala
---
@@ -57,15 +58,15 @@ object CurrentOrigin {
def reset
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11722#issuecomment-198640269
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11784#issuecomment-197916052
**[Test build #53432 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53432/consoleFull)**
for PR 11784 at commit
[`d263159`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11557#issuecomment-198136155
**[Test build #53480 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53480/consoleFull)**
for PR 11557 at commit
[`b87f2b8`](https://g
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11737#issuecomment-197735040
@jodersky there are still two failures here. I probably won't have time to
look at it until next week. If you have some time, please go for it! I think
some are legitimate
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11722#issuecomment-198640066
**[Test build #53599 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53599/consoleFull)**
for PR 11722 at commit
[`7f68967`](https://g
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/11786#issuecomment-198165733
merging 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 JoshRosen opened a pull request:
https://github.com/apache/spark/pull/11801
[SPARK-13990] Automatically pick serializer when caching RDDs
Building on the `SerializerManager` introduced in SPARK-13926/ #11755, this
patch Spark modifies Spark's BlockManager to use RDD's Cl
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11815#issuecomment-198639998
**[Test build #53604 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53604/consoleFull)**
for PR 11815 at commit
[`b04529b`](https://gi
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/11620#discussion_r56418520
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/GeneralizedLinearRegression.scala
---
@@ -178,7 +192,8 @@ class GeneralizedLinearRegression
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11748#issuecomment-198177509
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/11581#issuecomment-197553266
Got the LGTM from @mengxr offline. I'll merge this with master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11730
---
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 ena
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11701#issuecomment-198004371
**[Test build #53443 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53443/consoleFull)**
for PR 11701 at commit
[`0900b13`](https://gi
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11770#issuecomment-197591689
I'm merging this in 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 no
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/11379#issuecomment-197598917
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 an
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/11756#discussion_r56323218
--- Diff: python/pyspark/sql/readwriter.py ---
@@ -162,6 +162,14 @@ def json(self, path, schema=None):
(e.g. 00012)
*
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11834#issuecomment-198638010
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11708#issuecomment-197595897
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11804#issuecomment-198484446
**[Test build #53548 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53548/consoleFull)**
for PR 11804 at commit
[`97503f1`](https://gi
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/11759#discussion_r56611911
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlan.scala ---
@@ -218,48 +218,64 @@ abstract class SparkPlan extends QueryPlan[SparkPla
1 - 100 of 540 matches
Mail list logo