Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15249
**[Test build #66394 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66394/consoleFull)**
for PR 15249 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/15351
Hi, @hvanhovell .
Could you give some optinion about this backport when you have sometime?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user loneknightpy commented on the issue:
https://github.com/apache/spark/pull/15285
ping @tdas
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15318
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66393/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15318
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 dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/15357
Thank you, @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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/15318
**[Test build #66393 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66393/consoleFull)**
for PR 15318 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15102
**[Test build #66397 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66397/consoleFull)**
for PR 15102 at commit
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/15357
LGTM - 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 zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/15102#discussion_r82030620
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaSource.scala
---
@@ -0,0 +1,396 @@
+/*
+ * Licensed to the
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/15102#discussion_r82030589
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaSource.scala
---
@@ -0,0 +1,396 @@
+/*
+ * Licensed to the
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/15102#discussion_r82030575
--- Diff:
external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaSourceSuite.scala
---
@@ -0,0 +1,422 @@
+/*
+ * Licensed to the
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/15102#discussion_r82030607
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaSource.scala
---
@@ -0,0 +1,396 @@
+/*
+ * Licensed to the
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/15102#discussion_r82030626
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaSource.scala
---
@@ -0,0 +1,396 @@
+/*
+ * Licensed to the
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/15102#discussion_r82030551
--- Diff: docs/structured-streaming-kafka-integration.md ---
@@ -0,0 +1,239 @@
+---
+layout: global
+title: Structured Streaming + Kafka
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/15102#discussion_r82030561
--- Diff: docs/structured-streaming-kafka-integration.md ---
@@ -0,0 +1,239 @@
+---
+layout: global
+title: Structured Streaming + Kafka
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/15102#discussion_r82030538
--- Diff: docs/structured-streaming-kafka-integration.md ---
@@ -0,0 +1,239 @@
+---
+layout: global
+title: Structured Streaming + Kafka
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15102
**[Test build #66396 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66396/consoleFull)**
for PR 15102 at commit
Github user Yunni commented on the issue:
https://github.com/apache/spark/pull/15148
@jkbradley Take you time for the code review. :) I will be working on the
open dataset testing at the same time.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user Yunni commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r82027195
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/LSH.scala ---
@@ -0,0 +1,334 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15148
**[Test build #66395 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66395/consoleFull)**
for PR 15148 at commit
Github user Yunni commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r82027114
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/LSH.scala ---
@@ -0,0 +1,334 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
Github user Yunni commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r82027003
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/LSH.scala ---
@@ -0,0 +1,334 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
Github user Yunni commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r82027065
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/LSH.scala ---
@@ -0,0 +1,334 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
Github user Yunni commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r82027088
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/LSH.scala ---
@@ -0,0 +1,334 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
Github user Yunni commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r82026834
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/LSH.scala ---
@@ -0,0 +1,334 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/15335#discussion_r82010161
--- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala
---
@@ -1255,27 +1255,46 @@ class DAGScheduler(
s"longer
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/15335#discussion_r82021050
--- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala
---
@@ -1255,27 +1255,46 @@ class DAGScheduler(
s"longer
Github user Yunni commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r82018962
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/LSH.scala ---
@@ -0,0 +1,338 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15249
**[Test build #66394 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66394/consoleFull)**
for PR 15249 at commit
Github user squito commented on the issue:
https://github.com/apache/spark/pull/15249
I forgot to add that I had turned off blacklisting by default, I agree with
your suggestion Kay. I pushed another commit which updated the docs as well.
There are some other small style things and
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/15249#discussion_r82007301
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/TaskSetManagerSuite.scala ---
@@ -809,32 +816,65 @@ class TaskSetManagerSuite extends SparkFunSuite
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15009
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/15009
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66392/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15009
**[Test build #66392 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66392/consoleFull)**
for PR 15009 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/15318
Thank you, @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 feature
enabled and
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15318
**[Test build #66393 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66393/consoleFull)**
for PR 15318 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15318
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 the issue:
https://github.com/apache/spark/pull/15318
Will make a try. 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
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14531
@cloud-fan Hive does not copy the table properties in CREATE TABLE LIKE
---
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 gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15358#discussion_r82001717
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala
---
@@ -52,9 +52,15 @@ case class CatalogStorageFormat(
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15358
@rxin Sorry, I did not finish the PR description last night. The connection
is broken in the train. Will fix it soon.
---
If your project is set up for it, you can reply to this email and have
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/15249#discussion_r81999277
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSetManager.scala ---
@@ -592,34 +589,54 @@ private[spark] class TaskSetManager(
*
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15044
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66391/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15044
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/15044
**[Test build #66391 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66391/consoleFull)**
for PR 15044 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15246
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66390/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15246
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/15246
**[Test build #66390 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66390/consoleFull)**
for PR 15246 at commit
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/15359
I like @srowen's suggestion. Otherwise 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
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/15359#discussion_r81990605
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/NaiveBayes.scala ---
@@ -177,7 +177,7 @@ class NaiveBayes @Since("1.5.0") (
val
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14567
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66389/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14567
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/14963
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66387/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14567
**[Test build #66389 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66389/consoleFull)**
for PR 14567 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14963
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/14963
**[Test build #66387 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66387/consoleFull)**
for PR 14963 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15342
This is what SPARK-3261 is about. It's a corner case to be sure. To me it
seems like having duplicate centroids is worse because the model loses some of
its meaning. Points may arbitrarily assign to
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/15342
I'm more prefer to maintain the original logic that keep
```model.clusterCenters.length``` equal to ```k```. Was there some discussion
as to make this change?
I checked popular Python
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15009
**[Test build #66392 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66392/consoleFull)**
for PR 15009 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15343
@davies OK I think I'm 'fixing' this in the wrong place. RDD.setName() will
change the name displayed in the storage tab, but, this new Dataset.setName()
won't. You're right that it's always the
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/15009
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 wishes so,
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/15009
add to whitelist
---
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 zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/15314
@sethah I have add weighCol datatype checking into `checkNumericTypes`.
---
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 issue:
https://github.com/apache/spark/pull/15246
**[Test build #66390 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66390/consoleFull)**
for PR 15246 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15044
**[Test build #66391 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66391/consoleFull)**
for PR 15044 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15246
Jenkins 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
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15246
Jenkins add to whitelist
---
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 viirya commented on the issue:
https://github.com/apache/spark/pull/14912
ping @cloud-fan @hvanhovell @srinathshankar Can you take a look?
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14918
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66388/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14918
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/14918
**[Test build #66388 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66388/consoleFull)**
for PR 14918 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14567
**[Test build #66389 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66389/consoleFull)**
for PR 14567 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14963
**[Test build #66387 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66387/consoleFull)**
for PR 14963 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14918
**[Test build #66388 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66388/consoleFull)**
for PR 14918 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15361
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/15361
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66386/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15361
**[Test build #66386 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66386/consoleFull)**
for PR 15361 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15346
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66385/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15346
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/15346
**[Test build #66385 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66385/consoleFull)**
for PR 15346 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15361
**[Test build #66386 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66386/consoleFull)**
for PR 15361 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15361
@yhuai and @liancheng Do you mind if I ask to review 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
GitHub user HyukjinKwon opened a pull request:
https://github.com/apache/spark/pull/15361
[SPARK-17765][SQL] Support for writing out user-defined type in ORC
datasource
## What changes were proposed in this pull request?
`OrcStruct` is being created based on string
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15258
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/15258
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66384/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15258
**[Test build #66384 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66384/consoleFull)**
for PR 15258 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15354
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/15354
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66383/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15354
**[Test build #66383 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66383/consoleFull)**
for PR 15354 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15072
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66382/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15072
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/15072
**[Test build #66382 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66382/consoleFull)**
for PR 15072 at commit
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/15346#discussion_r81937555
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/SparkSqlParserSuite.scala
---
@@ -67,9 +86,133 @@ class SparkSqlParserSuite extends
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/15346
@hvanhovell Thank you for your suggestion! I've addressed your comment and
amended the testcases. Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/15346#discussion_r81936312
--- Diff:
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
@@ -593,6 +593,14 @@ colTypeList
;
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15341
Possibly @MLnick would have some thoughts on this one. I know there's a lot
going on here but mostly it's deletion and comments. The math changes are
probably best proven by the unit tests. I think
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15346
**[Test build #66385 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66385/consoleFull)**
for PR 15346 at commit
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/15324
I tend to make GaussianNB as a special `modelType` option in current NB.
However, there are significant differences:
1, 'theta' matrix is used to store means
2, extra `sigma` matrix
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/15102#discussion_r81930531
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaSource.scala
---
@@ -0,0 +1,396 @@
+/*
+ * Licensed to the Apache
301 - 400 of 502 matches
Mail list logo