Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13130
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 does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13130
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61142/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13130
**[Test build #61142 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61142/consoleFull)**
for PR 13130 at commit
[`82d78a3`](https://github.com/apache/spark/commit/
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/13778
ping @vlad17 @davies @liancheng Any thing else?
---
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 issue:
https://github.com/apache/spark/pull/13130
**[Test build #61142 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61142/consoleFull)**
for PR 13130 at commit
[`82d78a3`](https://github.com/apache/spark/commit/8
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/13130
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 this feature
enabled and wishes so,
Github user Div333 commented on the issue:
https://github.com/apache/spark/pull/13786
Hello everyone, Thanks a lot for implementing the pivot functionality. I
have started using SparkR recently and would like to know if the pivot method
is included in the library as I don't find it i
Github user JustinPihony commented on the issue:
https://github.com/apache/spark/pull/12601
Bump @HyukjinKwon I have some comments to your comments. Could you please
review them and I can push my changes.
---
If your project is set up for it, you can reply to this email and have your
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/13882
Please close this PR.
---
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,
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/13720#discussion_r68343766
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -522,7 +523,7 @@ case class DescribeTableCommand(table: TableIdent
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13882
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 project does not have this feat
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13865#discussion_r68343609
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/QueryPartitionSuite.scala ---
@@ -65,4 +68,77 @@ class QueryPartitionSuite extends QueryTest wi
GitHub user liu549676915 opened a pull request:
https://github.com/apache/spark/pull/13882
Branch 1.6
## What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
## How was this patch tested?
(Please explain how th
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/13720
For the test, currently we only have one `desc table` test in
`HiveDDLSuite`, It will be good if we can have an individual test suite for it.
---
If your project is set up for it, you can reply t
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13720#discussion_r68342801
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -522,7 +523,7 @@ case class DescribeTableCommand(table: TableI
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13874
**[Test build #61141 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61141/consoleFull)**
for PR 13874 at commit
[`ec0506f`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13880
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61139/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13880
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 does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13880
**[Test build #61139 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61139/consoleFull)**
for PR 13880 at commit
[`919f520`](https://github.com/apache/spark/commit/
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/13680
@kiszk we should definitely put zero into the corresponding field when set
null. It will be a little harder than `UnsafeRow`, as we need `setNullBoolean`,
`setNullInt`, etc. but it's still doable.
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/13832
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13832
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61138/
Test PASSed.
---
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/13832
Merging to master/2.0. 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
enabled a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13832
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 does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13832
**[Test build #61138 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61138/consoleFull)**
for PR 13832 at commit
[`e2a1e1e`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13876
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61137/
Test PASSed.
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13860
@hvanhovell Sure, will do it! It sounds like you also like the suggestion
by @cloud-fan . Let me do it too. Thanks!
---
If your project is set up for it, you can reply to this email and have you
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13876
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 does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13876
**[Test build #61137 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61137/consoleFull)**
for PR 13876 at commit
[`5a9f4ec`](https://github.com/apache/spark/commit/
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/13837#discussion_r68341391
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetOptions.scala
---
@@ -40,7 +40,7 @@ private[sql] class Parque
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13876
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61135/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13876
**[Test build #61135 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61135/consoleFull)**
for PR 13876 at commit
[`6180daf`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13876
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 does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13876
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61136/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13876
**[Test build #61136 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61136/consoleFull)**
for PR 13876 at commit
[`e0239a0`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13699
**[Test build #61140 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61140/consoleFull)**
for PR 13699 at commit
[`cf464a3`](https://github.com/apache/spark/commit/c
Github user sitalkedia commented on a diff in the pull request:
https://github.com/apache/spark/pull/13699#discussion_r68340566
--- Diff:
core/src/main/java/org/apache/spark/shuffle/sort/UnsafeShuffleWriter.java ---
@@ -122,6 +123,8 @@ public UnsafeShuffleWriter(
this.task
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13881
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 project does not have this feat
GitHub user smurching opened a pull request:
https://github.com/apache/spark/pull/13881
[SPARK-3723] [MLlib] Adding instrumentation to random forests
## What changes were proposed in this pull request?
In RandomForest.run(), added instrumentation for the number of node group
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/13877#discussion_r68337983
--- Diff: R/pkg/R/mllib.R ---
@@ -390,23 +376,41 @@ setMethod("predict", signature(object =
"KMeansModel"),
return(dataFrame(callJMethod(obje
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13794
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61133/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13794
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 does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13794
**[Test build #61133 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61133/consoleFull)**
for PR 13794 at commit
[`b8ddcdb`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13880
**[Test build #61139 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61139/consoleFull)**
for PR 13880 at commit
[`919f520`](https://github.com/apache/spark/commit/9
GitHub user rdblue opened a pull request:
https://github.com/apache/spark/pull/13880
SPARK-16178: Remove unnecessary Hive partition check.
## What changes were proposed in this pull request?
This removes a check that partition names match from the Hive write path,
which was
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13879
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61134/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13879
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 does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13879
**[Test build #61134 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61134/consoleFull)**
for PR 13879 at commit
[`63d280f`](https://github.com/apache/spark/commit/
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/13865#discussion_r68335825
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/TableReader.scala ---
@@ -230,10 +234,21 @@ class HadoopTableReader(
// Fill all par
Github user liancheng commented on the issue:
https://github.com/apache/spark/pull/13865
This is ready for review.
cc @yhuai @cloud-fan @clockfly
---
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 d
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/13860
@gatorsmile could you also remove the current uses of `new
MetadataBuilder().putString("comment", "test").build()` (and friends) from the
code base?
---
If your project is set up for it, you ca
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/13832
LGTM - pending jenkins.
---
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 wish
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13832
**[Test build #61138 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61138/consoleFull)**
for PR 13832 at commit
[`e2a1e1e`](https://github.com/apache/spark/commit/e
Github user sameeragarwal commented on a diff in the pull request:
https://github.com/apache/spark/pull/13832#discussion_r68334002
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ColumnVector.java
---
@@ -847,6 +862,11 @@ public final int appendStruct(bo
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13876
**[Test build #61137 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61137/consoleFull)**
for PR 13876 at commit
[`5a9f4ec`](https://github.com/apache/spark/commit/5
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/13870
Thank you for merging, @liancheng and @davies .
---
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 liancheng commented on the issue:
https://github.com/apache/spark/pull/13870
LGTM, merging to master and branch-2.0. 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 f
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/13870
---
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 issue:
https://github.com/apache/spark/pull/13876
**[Test build #61136 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61136/consoleFull)**
for PR 13876 at commit
[`e0239a0`](https://github.com/apache/spark/commit/e
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/11863
*1+2. Architecture*
I had to double check the code to understand the use of sorted list of
executors to get consistent topic-to-preferred locations. PLEASE WRITE MORE
SCALA DOCS on how the logic in
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13876
**[Test build #61135 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61135/consoleFull)**
for PR 13876 at commit
[`6180daf`](https://github.com/apache/spark/commit/6
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13879
**[Test build #61134 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61134/consoleFull)**
for PR 13879 at commit
[`63d280f`](https://github.com/apache/spark/commit/6
GitHub user hhbyyh opened a pull request:
https://github.com/apache/spark/pull/13879
[SPARK-16177] [ML] model loading backward compatibility for ml.regression
## What changes were proposed in this pull request?
jira: https://issues.apache.org/jira/browse/SPARK-16177
model loa
Github user bomeng commented on the issue:
https://github.com/apache/spark/pull/13720
@cloud-fan Is this one worth to be fixed?
---
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
en
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/13815
---
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 yhuai commented on the issue:
https://github.com/apache/spark/pull/13815
LGTM. Merging to master and branch 2.0.
---
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
enabl
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/13876#discussion_r68329169
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -793,6 +794,20 @@ object ConstantFolding extends
Github user davies commented on the issue:
https://github.com/apache/spark/pull/13878
cc @JoshRosen
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13794
**[Test build #61133 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61133/consoleFull)**
for PR 13794 at commit
[`b8ddcdb`](https://github.com/apache/spark/commit/b
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/13876#discussion_r68328505
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -793,6 +794,20 @@ object ConstantFolding extends Rule[Logi
Github user yinxusen commented on the issue:
https://github.com/apache/spark/pull/13794
test it 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
enabled and wishes so, or i
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/13876#discussion_r68328083
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -793,6 +794,20 @@ object ConstantFolding extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13878
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61132/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13878
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 does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13878
**[Test build #61132 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61132/consoleFull)**
for PR 13878 at commit
[`8d4b50b`](https://github.com/apache/spark/commit/
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/13876#discussion_r68327452
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -793,6 +794,20 @@ object ConstantFolding extends Rule[Logi
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/13876#discussion_r68327432
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -793,6 +794,20 @@ object ConstantFolding extends Rule[Logi
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/13872
Thank you, @mengxr , @liancheng , and @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 this
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/13872
---
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 mengxr commented on the issue:
https://github.com/apache/spark/pull/13872
Merged into master and branch-2.0. 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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13876
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 does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13876
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61126/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13876
**[Test build #61126 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61126/consoleFull)**
for PR 13876 at commit
[`e33b7f4`](https://github.com/apache/spark/commit/
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/13872
Thank you, @rxin . I hope so, too.
---
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
en
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/13872
I think it'd be difficult to guarantee ordering all the time, but since the
change is small we might as well take this tiny step to ensure stronger
compatibility.
---
If your project is set up for i
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/13872
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 and wishes so, or if the feature
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/13788#discussion_r68324868
--- Diff: python/pyspark/cloudpickle.py ---
@@ -169,7 +169,10 @@ def save_function(self, obj, name=None):
if name is None:
Github user JoshRosen commented on the issue:
https://github.com/apache/spark/pull/13788
Do you mind leaving a comment in the code referencing this issue / maybe
linking to that `six` issue so that it's easier for a reader to figure out
what's going on? Otherwise, this looks good to m
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/13788#discussion_r68324611
--- Diff: python/pyspark/cloudpickle.py ---
@@ -326,7 +329,10 @@ def save_global(self, obj, name=None,
pack=struct.pack):
modname = geta
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/13837#discussion_r68324497
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetOptions.scala
---
@@ -40,7 +40,7 @@ private[sql] class ParquetOptio
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/13788#discussion_r68324369
--- Diff: python/pyspark/cloudpickle.py ---
@@ -169,7 +169,10 @@ def save_function(self, obj, name=None):
if name is None:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13656
**[Test build #61131 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61131/consoleFull)**
for PR 13656 at commit
[`ed384c7`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13656
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61131/
Test FAILed.
---
Github user junyangq commented on a diff in the pull request:
https://github.com/apache/spark/pull/13877#discussion_r68324191
--- Diff: R/pkg/R/mllib.R ---
@@ -390,23 +376,41 @@ setMethod("predict", signature(object =
"KMeansModel"),
return(dataFrame(callJMethod(ob
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13878
**[Test build #61132 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61132/consoleFull)**
for PR 13878 at commit
[`8d4b50b`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13656
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 does not have this feature
e
Github user davies commented on the issue:
https://github.com/apache/spark/pull/13788
cc @JoshRosen
---
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
GitHub user davies opened a pull request:
https://github.com/apache/spark/pull/13878
[SPARK-16175] [PYSPARK] handle None for UDT
## What changes were proposed in this pull request?
Scala UDT will bypass all the null and will not pass them into serialize()
and deserialize()
Github user ahmed-mahran commented on a diff in the pull request:
https://github.com/apache/spark/pull/13866#discussion_r68323315
--- Diff:
streaming/src/test/scala/org/apache/spark/streaming/StreamingContextSuite.scala
---
@@ -327,6 +329,50 @@ class StreamingContextSuite extends
Github user ahmed-mahran commented on a diff in the pull request:
https://github.com/apache/spark/pull/13866#discussion_r68323120
--- Diff:
streaming/src/test/scala/org/apache/spark/streaming/StreamingContextSuite.scala
---
@@ -327,6 +329,50 @@ class StreamingContextSuite extends
101 - 200 of 481 matches
Mail list logo