Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18779
@maropu Because we already support it in Dataset API as the example
https://github.com/apache/spark/pull/18779#issuecomment-319880753 shows, I'm
afraid that there are users using this feature. If we
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/18834
With #18235 , I added Http(s) support for resources like files, jars.
SparkSubmit will fetch them remotely and download to local tmp dir.
---
If your project is set up for it, you can reply to
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/18834
I think https should be supported with this JIRA
(https://github.com/apache/spark/pull/18235).
---
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 issue:
https://github.com/apache/spark/pull/18834
**[Test build #80209 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80209/testReport)**
for PR 18834 at commit
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/16158#discussion_r131133998
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/tuning/CrossValidatorSuite.scala ---
@@ -66,6 +66,29 @@ class CrossValidatorSuite
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/16158
@hhbyyh sorry for the delay. Left a few review comments.
Tested the examples and it looks cool! Very useful
---
If your project is set up for it, you can reply to this email and have your
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/16158#discussion_r131133851
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tuning/TuningSummary.scala ---
@@ -0,0 +1,58 @@
+/*
+ * Licensed to the Apache Software
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/16158#discussion_r131133018
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tuning/CrossValidator.scala ---
@@ -133,7 +134,10 @@ class CrossValidator @Since("1.2.0")
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/16158#discussion_r131132815
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tuning/CrossValidator.scala ---
@@ -229,6 +233,29 @@ class CrossValidatorModel private[ml] (
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/16158#discussion_r131132649
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tuning/TrainValidationSplit.scala ---
@@ -231,6 +235,29 @@ class TrainValidationSplitModel private[ml] (
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/16158#discussion_r131133463
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tuning/TuningSummary.scala ---
@@ -0,0 +1,58 @@
+/*
+ * Licensed to the Apache Software
GitHub user srowen opened a pull request:
https://github.com/apache/spark/pull/18834
[SPARK-21618][DOCS] Note that http(s) not accepted in spark-submit jar uri
## What changes were proposed in this pull request?
Remove https from list of supported URIs, until it's
Github user skonto commented on the issue:
https://github.com/apache/spark/pull/18630
@vanzin fixed the issues. Please give it another try or merge.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18630
**[Test build #80208 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80208/testReport)**
for PR 18630 at commit
Github user highfei2011 commented on the issue:
https://github.com/apache/spark/pull/18807
Hi, @srowen @markhamstra ,This problem has been
solvedï¼[https://github.com/apache/spark/pull/18808](url)
---
If your project is set up for it, you can reply to this email and have your
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18804
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/18804
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80204/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18804
**[Test build #80204 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80204/testReport)**
for PR 18804 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17357
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/17357
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80202/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17357
**[Test build #80202 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80202/testReport)**
for PR 17357 at commit
Github user 2ooom commented on the issue:
https://github.com/apache/spark/pull/18783
Thank you @srowen. Seems like jenkins is happy. Should we merge in this
case?
---
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 MLnick commented on the issue:
https://github.com/apache/spark/pull/18313
I commented on the
[JIRA](https://issues.apache.org/jira/browse/SPARK-21086?focusedCommentId=16112623=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-16112623).
I
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18538#discussion_r131121892
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/evaluation/ClusteringEvaluatorSuite.scala
---
@@ -0,0 +1,235 @@
+/*
+ * Licensed to the Apache
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/18733#discussion_r131121125
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tuning/CrossValidator.scala ---
@@ -112,16 +112,16 @@ class CrossValidator @Since("1.2.0")
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/18797
I've figured out the problem, and pretty sure it's a problem in the AFT
test that was hidden until now. It runs AFTSurvivlaRegression on this input:
```
++-+--+--+
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18802
**[Test build #80207 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80207/testReport)**
for PR 18802 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18833
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80200/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18833
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 maropu commented on the issue:
https://github.com/apache/spark/pull/18779
@viirya btw, we still need to support group/sort-by-ordinal in Dataset? If
this pr merged, the behaviour seems to change. I feel the syntax in Dataset is
a little confusing..
---
If your project
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18833
**[Test build #80200 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80200/testReport)**
for PR 18833 at commit
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/18779
@viirya great! 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 and wishes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18779
**[Test build #80206 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80206/testReport)**
for PR 18779 at commit
Github user 10110346 commented on the issue:
https://github.com/apache/spark/pull/18779
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/18808
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/18808
**[Test build #3874 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3874/testReport)**
for PR 18808 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14151
**[Test build #80205 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80205/testReport)**
for PR 14151 at commit
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/18815
Yes, I am also considering this security issue. If this only shows a path,
only authorized people can log on to the spark cluster lunix server for log
view, which is relatively safe.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14151
**[Test build #80203 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80203/testReport)**
for PR 14151 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14151
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80203/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14151
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 srowen commented on the issue:
https://github.com/apache/spark/pull/18815
I don't know if this is that important, so not sure if it's worth much
code. Would people generally be able to access the logs anyway? is there a
security issue if I can see the masters logs and
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/18815
Ok, i will work it, just like hbase. Show master and worker logs, and
provide online viewing. Is this okay?
---
If your project is set up for it, you can reply to this email and have your
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/18797
The only number that is <= Double.PositiveInfinity is Double.NaN, because
it has no ordering at all with respect to anything. So init must be NaN somehow.
It's called from LBFGS in Breeze,
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18779
@maropu The PR is at #17770.
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/18815
OK, then how can this work? this won't render as a usable link.
---
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
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/18815
Yes, it is not an http URI ,at present only a path to tell the user log
path where to facilitate the lunix server query related log information.
---
If your project is set up for it, you
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18804
**[Test build #80204 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80204/testReport)**
for PR 18804 at commit
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/18281
I also think we can leave any potential improvements for parallelism on the
Python side (as well as the test side if we come up with a good way of testing
that fitting is actually being done in
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/18281
@ajaysaini725 Could you resolve merge conflicts and address the remaining
outstanding review comments?
I left a few minor additional comments. Overall I think this is just about
ready.
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/18281#discussion_r131104871
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/OneVsRestSuite.scala ---
@@ -101,6 +101,44 @@ class OneVsRestSuite extends SparkFunSuite
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/18281#discussion_r131104233
--- Diff: python/pyspark/ml/classification.py ---
@@ -1517,20 +1518,23 @@ class OneVsRest(Estimator, OneVsRestParams,
JavaMLReadable, JavaMLWritable):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14151
**[Test build #80203 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80203/testReport)**
for PR 14151 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18816
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/18815
It's not an http URI though right? it's a path. I'm missing why this is
browseable.
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/18816
Merged 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 wishes so, or
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/18281#discussion_r131102727
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/OneVsRest.scala ---
@@ -294,6 +296,18 @@ final class OneVsRest @Since("1.4.0") (
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/18829
@srowen @ajbozarth
Help review the code,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
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/18815
1.The master process information is as followsï¼
/usr/java8/jdk/bin/java -cp
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18668
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80198/
Test PASSed.
---
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/18538
@gatorsmile Could you help to trigger the test job? It seems I can't do it
now. Thanks.
---
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 issue:
https://github.com/apache/spark/pull/18668
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/18668
**[Test build #80198 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80198/testReport)**
for PR 18668 at commit
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/18538#discussion_r131100309
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/evaluation/ClusteringEvaluatorSuite.scala
---
@@ -0,0 +1,235 @@
+/*
+ * Licensed to the
Github user ScrapCodes commented on a diff in the pull request:
https://github.com/apache/spark/pull/14151#discussion_r131100257
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/HadoopFileWholeTextReader.scala
---
@@ -0,0 +1,57 @@
+/*
+ *
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18816
**[Test build #3876 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3876/testReport)**
for PR 18816 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18832
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80199/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18832
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 steveloughran commented on the issue:
https://github.com/apache/spark/pull/18628
Thanks for making sure this is consistent with other uses of
Configuration.get(); consistency is critical here
---
If your project is set up for it, you can reply to this email and have your
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18832
**[Test build #80199 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80199/testReport)**
for PR 18832 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/18815
Hm, I thought so, even in standalone mode, but I don't know.
OK, if it's a file URI, is that going to be browseable?
---
If your project is set up for it, you can reply to this email and have
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/18668#discussion_r131095350
--- Diff:
sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkSQLCLIDriver.scala
---
@@ -50,6 +50,7 @@ private[hive]
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18528
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/18528
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80201/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18528
**[Test build #80201 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80201/testReport)**
for PR 18528 at commit
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/18668#discussion_r131094720
--- Diff: docs/configuration.md ---
@@ -2335,5 +2335,61 @@ The location of these configuration files varies
across Hadoop versions, but
a common
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/18815
@srowen
Are you sure that the master and worker logs can be stored in hdfs?
Spark's master and worker logs are generated by log4j.
---
If your project is set up for it, you can
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/18668#discussion_r131093892
--- Diff: docs/configuration.md ---
@@ -2335,5 +2335,61 @@ The location of these configuration files varies
across Hadoop versions, but
a common
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/18668#discussion_r131093320
--- Diff: docs/configuration.md ---
@@ -2335,5 +2335,61 @@ The location of these configuration files varies
across Hadoop versions, but
a common
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18804
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/17357
**[Test build #80202 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80202/testReport)**
for PR 17357 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18804
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80196/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18779
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80197/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18779
**[Test build #80197 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80197/testReport)**
for PR 18779 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18804
**[Test build #80196 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80196/testReport)**
for PR 18804 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18779
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 maropu commented on the issue:
https://github.com/apache/spark/pull/18779
ok, 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 and wishes so, or if
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18528
**[Test build #80201 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80201/testReport)**
for PR 18528 at commit
Github user skonto commented on the issue:
https://github.com/apache/spark/pull/18528
@srowen fixed, ready for merge.
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/18811
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/18816
**[Test build #3876 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3876/testReport)**
for PR 18816 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/18745
Ping @caneGuy -- adding `[test-maven]` will let us also verify this passes
the Maven build
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/18833
Pg does not accept this query;
```
postgres=# select sqrt(3);
sqrt
--
1.73205080756888
(1 row)
postgres=# select sqrt(-1);
ERROR:
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/18811
merged 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 wishes so, or
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18779
@maropu I have a PR before to solve that. Due to some reasons it will be
merged on 2.3. I am out of laptop, will refer it once I can access laptop.
On Aug 3, 2017 5:07 PM,
Github user zuotingbing commented on the issue:
https://github.com/apache/spark/pull/18811
ok, have done. Thanks @srowen .
---
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 maropu commented on the issue:
https://github.com/apache/spark/pull/18779
@gatorsmile @viirya I looked into why it applied some analyzer rules into
already-analyzed plans and I noticed that some rules used
`transform/transformUp` instead of `resolveOperators` in `apply`.
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18607
@radford1 Have you added the test as suggested by @gatorsmile?
---
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
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/18832
@sethah
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18821
---
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
301 - 400 of 482 matches
Mail list logo