Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12353#issuecomment-211740746
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/12353#issuecomment-211740744
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/12353#issuecomment-211740602
**[Test build #56191 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56191/consoleFull)**
for PR 12353 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12381#issuecomment-211740371
**[Test build #56189 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56189/consoleFull)**
for PR 12381 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12381#issuecomment-211740457
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/12381#issuecomment-211740456
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/12492#issuecomment-211740393
**[Test build #56196 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56196/consoleFull)**
for PR 12492 at commit
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12472#issuecomment-211740321
We should be able to remove almost all the methods on
InternalAccumulators.scala, shouldn't we? All that includes create, createAll,
createShuffleReadAccums, ...
---
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/12460#discussion_r60175987
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -254,6 +251,21 @@ class SparkSqlAstBuilder extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/12472#discussion_r60175966
--- Diff: core/src/main/scala/org/apache/spark/scheduler/StageInfo.scala ---
@@ -36,7 +36,7 @@ class StageInfo(
val rddInfos: Seq[RDDInfo],
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12492#issuecomment-211739747
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 rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12472#discussion_r60175851
--- Diff: core/src/main/scala/org/apache/spark/scheduler/StageInfo.scala ---
@@ -36,7 +36,7 @@ class StageInfo(
val rddInfos: Seq[RDDInfo],
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12472#discussion_r60175871
--- Diff: core/src/main/scala/org/apache/spark/scheduler/StageInfo.scala ---
@@ -36,7 +36,7 @@ class StageInfo(
val rddInfos: Seq[RDDInfo],
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12492#issuecomment-211739754
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/12492#issuecomment-211739740
**[Test build #56195 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56195/consoleFull)**
for PR 12492 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/12472#discussion_r60175818
--- Diff: core/src/main/scala/org/apache/spark/executor/TaskMetrics.scala
---
@@ -217,21 +170,45 @@ class TaskMetrics private[spark] (initialAccums:
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12492#issuecomment-211739523
**[Test build #56195 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56195/consoleFull)**
for PR 12492 at commit
Github user ScrapCodes commented on the pull request:
https://github.com/apache/spark/pull/6848#issuecomment-211739456
IMO, this is useful in one way that hadoop configuration need not be a
global state. We can have a default set of configuration that we use everywhere
as a default.
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12472#discussion_r60175666
--- Diff: core/src/main/scala/org/apache/spark/executor/TaskMetrics.scala
---
@@ -268,23 +243,11 @@ private[spark] class ListenerTaskMetrics(
}
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12259#issuecomment-211739045
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/12492
[SPARK-12457] Fixed the Typos in Collection Functions
What changes were proposed in this pull request?
https://github.com/apache/spark/pull/12185 contains the original PR I
submitted
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12259#issuecomment-211739043
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/12259#issuecomment-211738829
**[Test build #56190 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56190/consoleFull)**
for PR 12259 at commit
Github user hujy commented on a diff in the pull request:
https://github.com/apache/spark/pull/12491#discussion_r60175227
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -159,7 +159,9 @@ private[classification] trait
Github user hujy commented on a diff in the pull request:
https://github.com/apache/spark/pull/12491#discussion_r60175230
--- Diff: python/pyspark/mllib/classification.py ---
@@ -262,6 +262,8 @@ def load(cls, sc, path):
model.setThreshold(threshold)
return
Github user gatorsmile closed the pull request at:
https://github.com/apache/spark/pull/10418
---
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 a diff in the pull request:
https://github.com/apache/spark/pull/12472#discussion_r60174901
--- Diff: core/src/main/scala/org/apache/spark/executor/TaskMetrics.scala
---
@@ -217,21 +170,45 @@ class TaskMetrics private[spark] (initialAccums:
Github user dibbhatt commented on the pull request:
https://github.com/apache/spark/pull/12484#issuecomment-211734038
Hi @JoshRosen , Isn't this fix is somehow related to the issue discussed
here https://github.com/apache/spark/pull/6990.
You can refer to the final comments
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12352#issuecomment-211733351
**[Test build #56194 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56194/consoleFull)**
for PR 12352 at commit
Github user lresende commented on the pull request:
https://github.com/apache/spark/pull/12270#issuecomment-211732862
Ok, I will work with @JoshRosen on the trigger part.
---
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 yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/12153#discussion_r60174480
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/sources/interfaces.scala ---
@@ -621,20 +621,40 @@ class HDFSFileCatalog(
def
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/12153#discussion_r60174434
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/sources/interfaces.scala ---
@@ -621,20 +621,40 @@ class HDFSFileCatalog(
def
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12472#discussion_r60174384
--- Diff: core/src/main/scala/org/apache/spark/TaskContextImpl.scala ---
@@ -36,15 +36,10 @@ private[spark] class TaskContextImpl(
override val
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12472#issuecomment-211732260
**[Test build #56193 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56193/consoleFull)**
for PR 12472 at commit
Github user gatorsmile closed the pull request at:
https://github.com/apache/spark/pull/11649
---
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 a diff in the pull request:
https://github.com/apache/spark/pull/12472#discussion_r60174365
--- Diff: core/src/main/scala/org/apache/spark/TaskContext.scala ---
@@ -65,7 +65,7 @@ object TaskContext {
* An empty task context that does not
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12472#issuecomment-211732276
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/12472#issuecomment-211732273
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 cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/12472#issuecomment-211731808
ready 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 rxin commented on the pull request:
https://github.com/apache/spark/pull/12270#issuecomment-211731818
We can run them via some trigger phrase though.
---
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 rxin commented on the pull request:
https://github.com/apache/spark/pull/12270#issuecomment-211731762
They have been flaky and causing other pull requests to fail. That's why we
shouldn't run them on Jenkins.
---
If your project is set up for it, you can reply to this
Github user lresende commented on the pull request:
https://github.com/apache/spark/pull/12270#issuecomment-211731480
@rxin Let me move them to a specific docker profile. But I would still run
them on Jenkins, as the infrastructure is already setup there.
---
If your project is set
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12472#issuecomment-211731220
**[Test build #56193 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56193/consoleFull)**
for PR 12472 at commit
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12270#issuecomment-211730704
I don't even think they should run on pull requests. Tests that require
extensive external setup (or downloading things) in general are flaky.
---
If your project is
Github user lresende commented on the pull request:
https://github.com/apache/spark/pull/12270#issuecomment-211729901
@rxin, just trying to understand, is the oracle test the only one failing ?
Or you are suggesting we move the whole docker based tests to a separate
profile that
Github user ScrapCodes commented on a diff in the pull request:
https://github.com/apache/spark/pull/12433#discussion_r60173470
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -260,7 +260,12 @@ private[spark] class BlockManager(
def
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12270#issuecomment-211729460
I just failed to build Spark locally once due to the docker oracle test.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12412#issuecomment-211729217
**[Test build #56192 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56192/consoleFull)**
for PR 12412 at commit
Github user holdenk commented on the pull request:
https://github.com/apache/spark/pull/12491#issuecomment-211729265
Thanks for taking the initiative on this PR - at first glance it seems like
this approach might not quite work but its easier to tell with some tests -
could you add a
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12270#issuecomment-211728923
These tests are too flaky. I've already seen a few failures.
We should disable them from the normal tests and maybe occasionally running
them (via some trigger or
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/12491#discussion_r60173022
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -159,7 +159,9 @@ private[classification] trait
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/12491#discussion_r60172784
--- Diff: python/pyspark/mllib/classification.py ---
@@ -262,6 +262,8 @@ def load(cls, sc, path):
model.setThreshold(threshold)
Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/12259#discussion_r60172565
--- Diff:
mllib/src/test/scala/org/apache/spark/mllib/linalg/udt/UDTSuite.scala ---
@@ -0,0 +1,99 @@
+/*
+ * Licensed to the Apache Software
Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/12259#discussion_r60172550
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/linalg/udt/VectorUDT.scala ---
@@ -0,0 +1,103 @@
+/*
+ * Licensed to the Apache Software
Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/12259#discussion_r60172539
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/linalg/udt/MatrixUDT.scala ---
@@ -0,0 +1,108 @@
+/*
+ * Licensed to the Apache Software
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12352#issuecomment-211726625
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/12352#issuecomment-211726623
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/12352#issuecomment-211726480
**[Test build #56185 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56185/consoleFull)**
for PR 12352 at commit
Github user dongjoon-hyun commented on the pull request:
https://github.com/apache/spark/pull/12353#issuecomment-211726421
Hi, @rxin . Now, the PR is updated in the following ways.
1. `CaseWhen` split into three classes:
- `CaseWhenBase`: abstract base class
-
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12059#issuecomment-211726085
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 viirya commented on the pull request:
https://github.com/apache/spark/pull/12259#issuecomment-211726110
Now the VectorUDT and MatrixUDT tests are moved to mllib.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12059#issuecomment-211726086
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/12059#issuecomment-211725948
**[Test build #56178 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56178/consoleFull)**
for PR 12059 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12259#issuecomment-211725741
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/12259#issuecomment-211725739
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/12353#issuecomment-211725663
**[Test build #56191 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56191/consoleFull)**
for PR 12353 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12259#issuecomment-211725565
**[Test build #56180 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56180/consoleFull)**
for PR 12259 at commit
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12352#discussion_r60171658
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileScanRDD.scala
---
@@ -53,33 +55,74 @@ class FileScanRDD(
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12409#issuecomment-211724615
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/12409#issuecomment-211724619
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user lw-lin commented on the pull request:
https://github.com/apache/spark/pull/12489#issuecomment-211724261
@zsxwing would you mind taking a look? 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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12409#issuecomment-211724245
**[Test build #56181 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56181/consoleFull)**
for PR 12409 at commit
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/12402#discussion_r60171349
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/clustering/GaussianMixture.scala ---
@@ -105,6 +108,15 @@ class GaussianMixtureModel private[ml] (
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12352#issuecomment-211722829
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/12352#issuecomment-211722828
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/12259#issuecomment-211722780
**[Test build #56190 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56190/consoleFull)**
for PR 12259 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12381#issuecomment-211722776
**[Test build #56189 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56189/consoleFull)**
for PR 12381 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12491#issuecomment-211722740
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12352#issuecomment-211722714
**[Test build #56183 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56183/consoleFull)**
for PR 12352 at commit
GitHub user hujy opened a pull request:
https://github.com/apache/spark/pull/12491
[SPARK-14712][ML]spark.ml.LogisticRegressionModel.toString should sumâ¦
## What changes were proposed in this pull request?
[SPARK-14712](https://issues.apache.org/jira/browse/SPARK-14712)
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12473#issuecomment-211722592
**[Test build #56188 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56188/consoleFull)**
for PR 12473 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12473#issuecomment-211722649
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/12473#issuecomment-211722647
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/12450#issuecomment-211721783
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/12450#issuecomment-211721784
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/12450#issuecomment-211721561
**[Test build #56175 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56175/consoleFull)**
for PR 12450 at commit
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12113#issuecomment-211721515
Looking at this now ... (might take a while though because i'd need to page
in all the details)
---
If your project is set up for it, you can reply to this email and
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12489#issuecomment-211721474
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/12489#issuecomment-211721473
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/12489#issuecomment-211721419
**[Test build #56184 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56184/consoleFull)**
for PR 12489 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12153#issuecomment-211721311
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/12153#issuecomment-211721313
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/12153#issuecomment-211721217
**[Test build #56158 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56158/consoleFull)**
for PR 12153 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12160#issuecomment-211720884
**[Test build #2827 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2827/consoleFull)**
for PR 12160 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12487#issuecomment-211720667
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/12487#issuecomment-211720669
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/12260#issuecomment-211720350
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 lianhuiwang commented on the pull request:
https://github.com/apache/spark/pull/12473#issuecomment-211720533
@jerryshao Yes, what you said is right.
---
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/12487#issuecomment-211720504
**[Test build #56176 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56176/consoleFull)**
for PR 12487 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12260#issuecomment-211720356
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
1 - 100 of 1099 matches
Mail list logo