Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18855
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
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18488
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
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18930#discussion_r133269222
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -2034,4 +2034,25 @@ class JsonSuite extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18940
**[Test build #80696 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80696/testReport)**
for PR 18940 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18942
You can patch it to your forked version.
---
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 issue:
https://github.com/apache/spark/pull/18943
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80692/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18943
**[Test build #80692 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80692/testReport)**
for PR 18943 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/18941
Can you write a proper PR title? It doesn't say what the change is about -
nor is it a complete sentence.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15770
**[Test build #80699 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80699/testReport)**
for PR 15770 at commit
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/18950#discussion_r133311882
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSetManager.scala ---
@@ -454,64 +477,68 @@ private[spark] class TaskSetManager(
}
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18855
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80697/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18488
**[Test build #80700 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80700/testReport)**
for PR 18488 at commit
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18887#discussion_r133318822
--- Diff: scalastyle-config.xml ---
@@ -86,7 +86,7 @@ This file is divided into 3 sections:
-
+
--- End diff --
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18849
**[Test build #80694 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80694/testReport)**
for PR 18849 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18940
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80691/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18940
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
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18942
@poplav This is not a regression from 2.0, right?
Since we might not release 2.1.2, this PR might not be merged to upstream
after a discussion with @zsxwing Maybe you can patch it in
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15770
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18940
**[Test build #80696 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80696/testReport)**
for PR 18940 at commit
Github user wangmiao1981 commented on the issue:
https://github.com/apache/spark/pull/15770
retest 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 wangmiao1981 commented on a diff in the pull request:
https://github.com/apache/spark/pull/15770#discussion_r133267575
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/clustering/PowerIterationClustering.scala
---
@@ -0,0 +1,213 @@
+/*
+ * Licensed to the
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/18952#discussion_r133268435
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/expressions/UserDefinedFunction.scala
---
@@ -101,7 +101,7 @@ case class UserDefinedFunction
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18519#discussion_r133294684
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/CoarseGrainedSchedulerBackend.scala
---
@@ -95,6 +99,12 @@ class
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/18943
I'm going to push this to avoid having to re-test it before pushing later
after more commits are checked in; I tested on the top of master now and there
are no new files that need fixing.
Github user ajbozarth commented on a diff in the pull request:
https://github.com/apache/spark/pull/18941#discussion_r133301952
--- Diff: core/src/main/scala/org/apache/spark/ui/UIUtils.scala ---
@@ -161,6 +161,7 @@ private[spark] object UIUtils extends Logging {
def
Github user MrBago commented on a diff in the pull request:
https://github.com/apache/spark/pull/18896#discussion_r133301688
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/LogisticRegressionSuite.scala
---
@@ -1392,6 +1415,61 @@ class LogisticRegressionSuite
Github user MrBago commented on the issue:
https://github.com/apache/spark/pull/18896
@jkbradley please take a look when you get a chance.
---
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 issue:
https://github.com/apache/spark/pull/18940
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80696/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18940
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18952
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80695/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18952
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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18943
---
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 vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18855#discussion_r133317789
--- Diff: core/src/test/scala/org/apache/spark/storage/DiskStoreSuite.scala
---
@@ -92,6 +92,45 @@ class DiskStoreSuite extends SparkFunSuite {
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18855#discussion_r133317809
--- Diff: core/src/test/scala/org/apache/spark/storage/DiskStoreSuite.scala
---
@@ -92,6 +92,45 @@ class DiskStoreSuite extends SparkFunSuite {
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18855#discussion_r133318267
--- Diff: core/src/test/scala/org/apache/spark/storage/DiskStoreSuite.scala
---
@@ -92,6 +92,45 @@ class DiskStoreSuite extends SparkFunSuite {
Github user dhruve commented on a diff in the pull request:
https://github.com/apache/spark/pull/18950#discussion_r133321125
--- Diff:
core/src/main/scala/org/apache/spark/ExecutorAllocationManager.scala ---
@@ -602,6 +604,21 @@ private[spark] class ExecutorAllocationManager(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18930#discussion_r133270299
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -2034,4 +2034,25 @@ class JsonSuite extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18855
**[Test build #80697 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80697/testReport)**
for PR 18855 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18943
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18952
**[Test build #80695 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80695/testReport)**
for PR 18952 at commit
Github user squito commented on the issue:
https://github.com/apache/spark/pull/18874
@srowen you have a good point about a case that becomes worse after this
change. Still I think this change is better on balance.
btw, there are more even more odd cases with dynamic
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15770
**[Test build #80698 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80698/testReport)**
for PR 15770 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15770
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80698/
Test FAILed.
---
Github user wangmiao1981 commented on the issue:
https://github.com/apache/spark/pull/15770
weird. Local style test passed. Anyway, I changed the order as required by
Jenkins.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18855
**[Test build #80697 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80697/testReport)**
for PR 18855 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15770
**[Test build #80702 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80702/testReport)**
for PR 15770 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18887
**[Test build #80701 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80701/testReport)**
for PR 18887 at commit
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/18538#discussion_r133360284
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/evaluation/ClusteringEvaluator.scala
---
@@ -0,0 +1,240 @@
+/*
+ * Licensed to the Apache
GitHub user caneGuy opened a pull request:
https://github.com/apache/spark/pull/18957
[SPARK-21744][CORE] Add retry logic for new broadcast in BroadcastManager
## What changes were proposed in this pull request?
When driver submit new stage and there is a bad disk before
Github user viirya closed the pull request at:
https://github.com/apache/spark/pull/18947
---
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 viirya commented on the issue:
https://github.com/apache/spark/pull/18947
Thanks @gatorsmile @dongjoon-hyun
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15770
**[Test build #80704 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80704/testReport)**
for PR 15770 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/12646
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18488
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18488
**[Test build #80700 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80700/testReport)**
for PR 18488 at commit
Github user janewangfb commented on a diff in the pull request:
https://github.com/apache/spark/pull/18492#discussion_r133355548
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/SparkListener.scala ---
@@ -291,6 +297,16 @@ private[spark] trait SparkListenerInterface {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18953
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
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/18954
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 this feature
enabled and
GitHub user tejasapatil opened a pull request:
https://github.com/apache/spark/pull/18954
[SPARK-17654] [SQL] Enable creating hive bucketed tables
## What changes were proposed in this pull request?
### Semantics:
- If the Hive table is bucketed, then INSERT node expect
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18492
**[Test build #80712 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80712/testReport)**
for PR 18492 at commit
GitHub user viirya opened a pull request:
https://github.com/apache/spark/pull/18956
[SPARK-21726][SQL] Check for structural integrity of the plan in Optimzer
in test mode.
## What changes were proposed in this pull request?
We have many optimization rules now in
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #80718 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80718/testReport)**
for PR 18956 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18953
**[Test build #80710 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80710/testReport)**
for PR 18953 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18955
cc @gengliangwang @sameeragarwal @hvanhovell
---
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 cloud-fan opened a pull request:
https://github.com/apache/spark/pull/18955
[SPARK-21743][SQL] top-most limit should not cause memory leak
## What changes were proposed in this pull request?
For top-most limit, we will use a special operator to execute it:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18855
**[Test build #80716 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80716/testReport)**
for PR 18855 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18955#discussion_r133359698
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -2658,4 +2658,9 @@ class SQLQuerySuite extends QueryTest with
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #80715 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80715/testReport)**
for PR 18956 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18957
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
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/18640
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 HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18926
The current codes around what this PR changes look not quite clean to me
too and we should clean around this.
But I think this PR itself is quite well-formed with the fix that is valid,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18492
**[Test build #80714 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80714/testReport)**
for PR 18492 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18955
**[Test build #80713 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80713/testReport)**
for PR 18955 at commit
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/18538#discussion_r133360674
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/evaluation/ClusteringEvaluatorSuite.scala
---
@@ -0,0 +1,225 @@
+/*
+ * Licensed to the
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18926
To be honest, the current codes do not look good to me. Since this does not
make the code worse, I will not revert it back.
---
If your project is set up for it, you can reply to this email and
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/18640
Thank you so much, @rxin , @cloud-fan , @sameeragarwal , @mridulm , @viirya
!
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18953
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80707/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18953
**[Test build #80707 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80707/testReport)**
for PR 18953 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18954
**[Test build #80711 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80711/testReport)**
for PR 18954 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #80717 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80717/testReport)**
for PR 18956 at commit
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18956#discussion_r133360047
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -37,6 +37,12 @@ import org.apache.spark.sql.types._
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18956#discussion_r133360995
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -37,6 +37,12 @@ import org.apache.spark.sql.types._
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18952
---
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 justinuang commented on the issue:
https://github.com/apache/spark/pull/15009
That would be incredible. Launching a new jvm and loading all of hadoop
takes about 4 seconds extra each time, versus reusing the launcher jvm, which
is really significant for us since we launch
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18951
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18951
**[Test build #80705 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80705/testReport)**
for PR 18951 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18951
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80705/
Test PASSed.
---
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/18896
LGTM except for making the test's title more descriptive. 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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/18953
**[Test build #80707 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80707/testReport)**
for PR 18953 at commit
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/18798
Merged into master, thanks for all.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16763
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16763
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80709/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16763
**[Test build #80709 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80709/testReport)**
for PR 16763 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/18640
Hi, @cloud-fan , @rxin , @sameeragarwal and @mridulm .
Could you merge this PR?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18952
Thanks! 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 feature
enabled and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12646
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12646
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80706/
Test FAILed.
---
GitHub user dongjoon-hyun opened a pull request:
https://github.com/apache/spark/pull/18953
[SPARK-20682][SQL] Implement new ORC data source based on Apache ORC
## What changes were proposed in this pull request?
Since #17924, #17943, and #17980 are a little large PRs, this
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/18902
@hhbyyh I rewrite the impl, and now all `NaN` and `missingValue` will be
transform to `null` at first, then current methods are used.
For columns only containing `null`, `null` is
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/17373#discussion_r133324363
--- Diff: mllib/src/main/scala/org/apache/spark/ml/ann/Layer.scala ---
@@ -527,9 +550,21 @@ private[ml] class FeedForwardModel private(
101 - 200 of 484 matches
Mail list logo