Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15429
**[Test build #66732 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66732/consoleFull)**
for PR 15429 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14847
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66730/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14847
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/14847
**[Test build #66730 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66730/consoleFull)**
for PR 14847 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14788
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/14788
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66728/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14788
**[Test build #66728 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66728/consoleFull)**
for PR 14788 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15432
**[Test build #66738 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66738/consoleFull)**
for PR 15432 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15382
**[Test build #3322 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3322/consoleFull)**
for PR 15382 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15428
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66735/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15432
**[Test build #66737 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66737/consoleFull)**
for PR 15432 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15428
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/15428
**[Test build #66735 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66735/consoleFull)**
for PR 15428 at commit
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/9
We should also just check save / load is backward compatible with older
versions. It should be, but subtle things can sneak in so let's be careful
about that.
---
If your project is set up for it,
GitHub user HyukjinKwon opened a pull request:
https://github.com/apache/spark/pull/15432
[SPARK-17854][SQL] rand/randn allows null as input seed
## What changes were proposed in this pull request?
This PR proposes `rand`/`randn` accept `null` as input. In this case, it
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/9#discussion_r82751668
--- Diff: mllib/src/main/scala/org/apache/spark/ml/util/ReadWrite.scala ---
@@ -446,6 +463,20 @@ private[ml] object DefaultParamsReader {
val cls =
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/9#discussion_r82753706
--- Diff: mllib/src/main/scala/org/apache/spark/ml/util/ReadWrite.scala ---
@@ -300,15 +301,23 @@ private[ml] object DefaultParamsWriter {
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/9#discussion_r82752375
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/clustering/KMeansSuite.scala ---
@@ -137,18 +143,64 @@ class KMeansSuite extends SparkFunSuite with
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/9#discussion_r82750575
--- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/KMeans.scala
---
@@ -333,13 +372,44 @@ class KMeans @Since("1.5.0") (
override def
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/9#discussion_r82749390
--- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/KMeans.scala
---
@@ -303,6 +312,20 @@ class KMeans @Since("1.5.0") (
@Since("1.5.0")
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/9#discussion_r82749322
--- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/KMeans.scala
---
@@ -303,6 +312,20 @@ class KMeans @Since("1.5.0") (
@Since("1.5.0")
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/9#discussion_r82749661
--- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/KMeans.scala
---
@@ -81,6 +81,13 @@ private[clustering] trait KMeansParams extends Params
with
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15431
**[Test build #66736 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66736/consoleFull)**
for PR 15431 at commit
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/12930
Move the new fix to #15431 which leverages ```RFormula forceIndexLabel```,
more succinct.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user yanboliang closed the pull request at:
https://github.com/apache/spark/pull/12930
---
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 yanboliang opened a pull request:
https://github.com/apache/spark/pull/15431
[SPARK-15153] [ML] [SparkR] Fix SparkR spark.naiveBayes error when label is
numeric type
## What changes were proposed in this pull request?
Fix SparkR ```spark.naiveBayes``` error when
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/15423#discussion_r82750631
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -168,17 +168,7 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/15423#discussion_r82750409
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -168,17 +168,7 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15428
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/15428
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66733/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15428
**[Test build #66733 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66733/consoleFull)**
for PR 15428 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15285
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/15285
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66725/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15285
**[Test build #66725 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66725/consoleFull)**
for PR 15285 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15430
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/15430
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66734/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15430
**[Test build #66734 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66734/consoleFull)**
for PR 15430 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15342
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66729/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15342
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/15342
**[Test build #66729 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66729/consoleFull)**
for PR 15342 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15388
Thanks! @rxin @cloud-fan @hvanhovell @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 not have this
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/9
I have a few high level questions on this:
Params
Why are we only setting` k` based on the `initialModel`? I had thought from
previous discussion above (it was a while ago now)
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15295
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/15295
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66722/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15295
**[Test build #66722 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66722/consoleFull)**
for PR 15295 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15427
cc @cloud-fan @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 feature
enabled and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15427
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66724/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15427
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/15428
**[Test build #66735 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66735/consoleFull)**
for PR 15428 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15427
**[Test build #66724 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66724/consoleFull)**
for PR 15427 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/15423#discussion_r82745403
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -168,17 +168,7 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/9
I misread DB's meaning in my previous comment.
I agree that the parameter settings of `initialModel`, if set, should take
precedence. If it conflicts with an existing `k` then log a warning.
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/13194
Resolved merge conflicts, ping @holdenk @MLnick @srowen @jkbradley to take
a look when you available. This is just add Python API and should be very
straight forward to move ahead. Thanks.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15430
**[Test build #66734 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66734/consoleFull)**
for PR 15430 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15423#discussion_r82744230
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -168,17 +168,7 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15405
**[Test build #3323 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3323/consoleFull)**
for PR 15405 at commit
GitHub user yanboliang opened a pull request:
https://github.com/apache/spark/pull/15430
[SPARK-15957][Follow-up][ML][PySpark] Add Python API for RFormula
forceIndexLabel.
## What changes were proposed in this pull request?
Follow-up work of #13675, add Python API for
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/15072#discussion_r82743571
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -53,7 +53,15 @@ import org.apache.spark.util.Utils
private[sql]
Github user witgo commented on a diff in the pull request:
https://github.com/apache/spark/pull/15297#discussion_r82743168
--- Diff: core/src/main/scala/org/apache/spark/shuffle/ShuffleManager.scala
---
@@ -48,7 +48,8 @@ private[spark] trait ShuffleManager {
handle:
Github user VinceShieh commented on a diff in the pull request:
https://github.com/apache/spark/pull/15428#discussion_r82743072
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/Bucketizer.scala
---
@@ -73,15 +78,27 @@ final class Bucketizer @Since("1.4.0")
Github user witgo commented on a diff in the pull request:
https://github.com/apache/spark/pull/15297#discussion_r82742902
--- Diff: core/src/main/scala/org/apache/spark/MapOutputTracker.scala ---
@@ -687,18 +691,21 @@ private[spark] object MapOutputTracker extends
Logging {
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14847
@viirya can you try to create a new operator for this optimization and make
it work with whole-stage-codegen? thanks!
---
If your project is set up for it, you can reply to this email and have
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15388
---
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 witgo commented on a diff in the pull request:
https://github.com/apache/spark/pull/15297#discussion_r82742056
--- Diff: core/src/main/scala/org/apache/spark/MapOutputTracker.scala ---
@@ -138,13 +138,16 @@ private[spark] abstract class MapOutputTracker(conf:
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15388
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 srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15428#discussion_r82741203
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/param/shared/sharedParams.scala ---
@@ -270,10 +270,10 @@ private[ml] trait HasFitIntercept extends Params
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15428#discussion_r82741770
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/Bucketizer.scala
---
@@ -128,8 +145,9 @@ object Bucketizer extends
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15428#discussion_r82741459
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/Bucketizer.scala
---
@@ -73,15 +78,27 @@ final class Bucketizer @Since("1.4.0") (@Since("1.4.0")
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15428
**[Test build #66733 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66733/consoleFull)**
for PR 15428 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15429
**[Test build #66732 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66732/consoleFull)**
for PR 15429 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15425
**[Test build #3321 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3321/consoleFull)**
for PR 15425 at commit
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/14847
This would really only be interesting if it works with whole stage code
gen; otherwise it is not really interesting.
In addition, it'd make sense to have an explicit operator for this, e.g.
GitHub user srowen opened a pull request:
https://github.com/apache/spark/pull/15429
[SPARK-17840] [DOCS] Add some pointers for wiki/CONTRIBUTING.md in
README.md and some warnings in PULL_REQUEST_TEMPLATE
## What changes were proposed in this pull request?
Link to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15428
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66731/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15428
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/15428
**[Test build #66731 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66731/consoleFull)**
for PR 15428 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15428
**[Test build #66731 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66731/consoleFull)**
for PR 15428 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15382
**[Test build #3322 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3322/consoleFull)**
for PR 15382 at commit
GitHub user VinceShieh opened a pull request:
https://github.com/apache/spark/pull/15428
[SPARK-17219][ML] enchanced NaN value handling in Bucketizer
## What changes were proposed in this pull request?
This PR is an enhancement of PR with commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15342
**[Test build #66729 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66729/consoleFull)**
for PR 15342 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14847
**[Test build #66730 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66730/consoleFull)**
for PR 14847 at commit
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/15072#discussion_r82738407
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -53,7 +53,15 @@ import org.apache.spark.util.Utils
private[sql]
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15342
Yeah, but now we have yet 2 more overloads. I had intended to point people
to 1 new overload, but I guess it's weird to make people specify the seed arg.
And optional args, the normal solution,
GitHub user viirya reopened a pull request:
https://github.com/apache/spark/pull/14847
[SPARK-17254][SQL] Filter can stop when the condition is false if the child
output is sorted
## What changes were proposed in this pull request?
From
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/14847
Re-open it and see if we can have some consensus about this direction.
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14702
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/14702
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66723/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14702
**[Test build #66723 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66723/consoleFull)**
for PR 14702 at commit
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/15072#discussion_r82737858
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -53,7 +53,15 @@ import org.apache.spark.util.Utils
private[sql]
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15426
---
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 dbtsai commented on the issue:
https://github.com/apache/spark/pull/9
+1 on what @sethah proposed. We can log with warn when k is modified by
setting the initial model. Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11459
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66726/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11459
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 rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15408#discussion_r82737475
--- Diff:
core/src/main/java/org/apache/spark/io/NioBufferedFileInputStream.java ---
@@ -0,0 +1,129 @@
+/*
+ * Licensed under the Apache License,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11459
**[Test build #66726 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66726/consoleFull)**
for PR 11459 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15426
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 wishes
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15426
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
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15408#discussion_r82737052
--- Diff:
core/src/main/java/org/apache/spark/io/NioBufferedFileInputStream.java ---
@@ -0,0 +1,129 @@
+/*
+ * Licensed under the Apache License,
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/14788#discussion_r82737073
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -2374,14 +2374,14 @@ object functions {
* @group datetime_funcs
*
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14788
**[Test build #66728 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66728/consoleFull)**
for PR 14788 at commit
501 - 600 of 661 matches
Mail list logo