Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15090#discussion_r80629937
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeColumnCommand.scala
---
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/15090#discussion_r80629684
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -87,19 +87,27 @@ class SparkSqlAstBuilder(conf: SQLConf) extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15090#discussion_r80629335
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeColumnCommand.scala
---
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15090#discussion_r80629224
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -87,19 +87,27 @@ class SparkSqlAstBuilder(conf: SQLConf) exten
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15090#discussion_r80628790
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeRow.java
---
@@ -31,6 +31,7 @@
import com.esotericsoftware.kryo.
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/12601#discussion_r80628570
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcRelationProvider.scala
---
@@ -19,37 +19,102 @@ package org.apache.
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/12601#discussion_r80628287
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcRelationProvider.scala
---
@@ -19,37 +19,102 @@ package org.apache
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15242
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65949/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15242
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15242
**[Test build #65949 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65949/consoleFull)**
for PR 15242 at commit
[`85cda01`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15255
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15255
**[Test build #65946 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65946/consoleFull)**
for PR 15255 at commit
[`741d59c`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15255
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65946/
Test PASSed.
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/12601#discussion_r80627940
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcRelationProvider.scala
---
@@ -19,37 +19,102 @@ package org.apache.s
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15090
**[Test build #65950 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65950/consoleFull)**
for PR 15090 at commit
[`7cd8f14`](https://github.com/apache/spark/commit/7
Github user danix800 closed the pull request at:
https://github.com/apache/spark/pull/15243
---
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 e
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/15090#discussion_r80626981
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeColumnCommand.scala
---
@@ -0,0 +1,179 @@
+/*
+ * Licensed to the Apac
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15255
... Hit one more bug in the write path...
---
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 featu
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/13680
---
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 ena
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/13680
thanks for your great work! 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 fe
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15251
---
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 ena
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15251
Thanks - merging in 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 wishe
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15242
**[Test build #65949 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65949/consoleFull)**
for PR 15242 at commit
[`85cda01`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15251
**[Test build #3290 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3290/consoleFull)**
for PR 15251 at commit
[`dbaf93c`](https://github.com/apache/spark/commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/15168
Hi, @hvanhovell .
Could you review this PR again, 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 do
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15252
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65945/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15252
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15252
**[Test build #65945 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65945/consoleFull)**
for PR 15252 at commit
[`952ef99`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15242
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65947/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15242
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15242
**[Test build #65947 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65947/consoleFull)**
for PR 15242 at commit
[`88c1069`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14897
**[Test build #65948 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65948/consoleFull)**
for PR 14897 at commit
[`2732531`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15253
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15253
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65944/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15253
**[Test build #65944 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65944/consoleFull)**
for PR 15253 at commit
[`fafdf43`](https://github.com/apache/spark/commit/
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15248#discussion_r80623063
--- Diff:
core/src/main/scala/org/apache/spark/status/api/v1/ApplicationListResource.scala
---
@@ -32,7 +32,14 @@ private[v1] class ApplicationListResource(u
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15248#discussion_r80623021
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/HistoryServer.scala ---
@@ -178,6 +178,23 @@ class HistoryServer(
provider.getListing()
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15246
Hm, I see why this happens to work, because the file is not packaged inside
a jar file. Normally that's what getResource is for, and if it were in a jar
this wouldn't work. It's not a bad idea though
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15252
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65943/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15252
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15252
**[Test build #65943 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65943/consoleFull)**
for PR 15252 at commit
[`1124edf`](https://github.com/apache/spark/commit/
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15255
... Hit a bug in the write path... Need to fix it at first...
---
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 rxin commented on the issue:
https://github.com/apache/spark/pull/15254
cc @JoshRosen and @davies
---
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/15255
Also need to add a test case for data source tables. Will do it later
---
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 proje
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15250
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65942/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15250
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15250
**[Test build #65942 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65942/consoleFull)**
for PR 15250 at commit
[`b125f2f`](https://github.com/apache/spark/commit/
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15242
Let me update the PR description just in 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 project does not have this
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15242
**[Test build #65947 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65947/consoleFull)**
for PR 15242 at commit
[`88c1069`](https://github.com/apache/spark/commit/8
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15242
@srowen I just took a scan twice and I think they should be all.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project d
Github user lins05 commented on the issue:
https://github.com/apache/spark/pull/15254
I guess we can also remove another workaround
[here](https://github.com/apache/spark/blob/v2.0.0/python/pyspark/rdd.py#L2320-L2328)
?
---
If your project is set up for it, you can reply to this ema
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15255
**[Test build #65946 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65946/consoleFull)**
for PR 15255 at commit
[`741d59c`](https://github.com/apache/spark/commit/7
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/15255
[SPARK-17680] [SQL] [TEST] Added a Testcase for Verifying Unicode Character
Support for Column Names and Comments
### What changes were proposed in this pull request?
When the version of the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15090
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15090
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65941/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15090
**[Test build #65941 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65941/consoleFull)**
for PR 15090 at commit
[`08df669`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15254
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feat
Github user JasonMWhite commented on the issue:
https://github.com/apache/spark/pull/15254
@davies you authored https://github.com/apache/spark/pull/5570 and reported
the issue in Py4J https://github.com/bartdag/py4j/issues/160. I happened across
this while spelunking through Py4J cod
GitHub user JasonMWhite opened a pull request:
https://github.com/apache/spark/pull/15254
[SPARK-17679] [PYSPARK] remove unnecessary Py4J ListConverter patch
## What changes were proposed in this pull request?
This PR removes a patch on ListConverter from
https://github.com
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/15249
This is awesome to separate this out. I should have time to review this
tomorrow and then hopefully we can (finally) merge this in the next few days!
---
If your project is set up for it, yo
Github user markhamstra commented on the issue:
https://github.com/apache/spark/pull/15213
@scwf I understand that you were trying to make the least invasive fix
possible to deal with the problem. That's usually a good thing to do, but even
when that kind of fix is getting to the roo
Github user koeninger commented on the issue:
https://github.com/apache/spark/pull/13998
I ran that test 100 times locally w/out error... you have any suggestions
on repro?
On Mon, Sep 26, 2016 at 6:40 PM, Cody Koeninger wrote:
> Sure I'll give it another look
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14079
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14079
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65939/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14079
**[Test build #65939 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65939/consoleFull)**
for PR 14079 at commit
[`278fff3`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15249
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65938/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15249
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15249
**[Test build #65938 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65938/consoleFull)**
for PR 15249 at commit
[`882b385`](https://github.com/apache/spark/commit/
Github user koeninger commented on a diff in the pull request:
https://github.com/apache/spark/pull/15102#discussion_r80617904
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaSource.scala
---
@@ -0,0 +1,344 @@
+/*
+ * Licensed to the Apa
Github user gatorsmile closed the pull request at:
https://github.com/apache/spark/pull/12618
---
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 gatorsmile commented on the issue:
https://github.com/apache/spark/pull/12618
The changes in this PR becomes completely out-of-dated. Will resubmit a new
PR for it. Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on Git
Github user koeninger commented on the issue:
https://github.com/apache/spark/pull/15102
Ok, finished a line-by-line compare + comment.
The biggest thing I'm having trouble reconciling is the stated emphasis on
limiting user options in order to give guarantees, yet throwing th
Github user koeninger commented on a diff in the pull request:
https://github.com/apache/spark/pull/15102#discussion_r80617145
--- Diff: external/kafka-0-10-sql/pom.xml ---
@@ -0,0 +1,82 @@
+
+
+
+http://maven.apache.org/POM/4.0.0";
xmlns:xsi="http://www.w3.org/2001
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15221
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15221
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65940/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15221
**[Test build #65940 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65940/consoleFull)**
for PR 15221 at commit
[`f2f1252`](https://github.com/apache/spark/commit/
Github user koeninger commented on a diff in the pull request:
https://github.com/apache/spark/pull/15102#discussion_r80617036
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaSourceRDD.scala
---
@@ -0,0 +1,163 @@
+/*
+ * Licensed to the
Github user koeninger commented on a diff in the pull request:
https://github.com/apache/spark/pull/15102#discussion_r80616878
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaSourceRDD.scala
---
@@ -0,0 +1,163 @@
+/*
+ * Licensed to the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15252
**[Test build #65945 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65945/consoleFull)**
for PR 15252 at commit
[`952ef99`](https://github.com/apache/spark/commit/9
Github user scwf commented on the issue:
https://github.com/apache/spark/pull/15213
@markhamstra in my fix i just want to make the minor changes for the
dagscheduer, and your fix is also ok to me, i can update this according your
comment. Thanks:)
/cc @zsxwing may also have commen
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15253
**[Test build #65944 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65944/consoleFull)**
for PR 15253 at commit
[`fafdf43`](https://github.com/apache/spark/commit/f
GitHub user jerryshao opened a pull request:
https://github.com/apache/spark/pull/15253
[SPARK-17678][REPL][Branch-1.6] Honor spark.replClassServer.port in
scala-2.11 repl
## What changes were proposed in this pull request?
Spark 1.6 Scala-2.11 repl doesn't honor "spark.rep
Github user koeninger commented on a diff in the pull request:
https://github.com/apache/spark/pull/15102#discussion_r80616386
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaSourceRDD.scala
---
@@ -0,0 +1,163 @@
+/*
+ * Licensed to the
Github user koeninger commented on a diff in the pull request:
https://github.com/apache/spark/pull/15102#discussion_r80616098
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaSourceRDD.scala
---
@@ -0,0 +1,163 @@
+/*
+ * Licensed to the
Github user cjjnjust commented on a diff in the pull request:
https://github.com/apache/spark/pull/15172#discussion_r80615974
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/sasl/aes/SparkAesCipher.java
---
@@ -0,0 +1,270 @@
+/*
+ * Licensed to the A
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15251
**[Test build #3290 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3290/consoleFull)**
for PR 15251 at commit
[`dbaf93c`](https://github.com/apache/spark/commit/
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15251
Jenkins, test this please.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user koeninger commented on a diff in the pull request:
https://github.com/apache/spark/pull/15102#discussion_r80615842
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaSourceProvider.scala
---
@@ -0,0 +1,263 @@
+/*
+ * Licensed to
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15252
cc @hvanhovell for review. This is mostly simple copy/paste. I did look
over the warnings in IntelliJ and spacing issues.
Also @yhuai and @hvanhovell - I noticed there was virtually 0 unit test
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15252
**[Test build #65943 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65943/consoleFull)**
for PR 15252 at commit
[`1124edf`](https://github.com/apache/spark/commit/1
GitHub user rxin opened a pull request:
https://github.com/apache/spark/pull/15252
[SPARK-17677][SQL] Break WindowExec.scala into multiple files
## What changes were proposed in this pull request?
As of Spark 2.0, all the window function execution code are in
WindowExec.scala. T
Github user koeninger commented on a diff in the pull request:
https://github.com/apache/spark/pull/15102#discussion_r80615307
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaSourceProvider.scala
---
@@ -0,0 +1,263 @@
+/*
+ * Licensed to
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/15090#discussion_r80615261
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeColumnCommand.scala
---
@@ -0,0 +1,179 @@
+/*
+ * Licensed to the
Github user koeninger commented on a diff in the pull request:
https://github.com/apache/spark/pull/15102#discussion_r80614899
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaSourceProvider.scala
---
@@ -0,0 +1,263 @@
+/*
+ * Licensed to
Github user koeninger commented on a diff in the pull request:
https://github.com/apache/spark/pull/15102#discussion_r80614481
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaSourceProvider.scala
---
@@ -0,0 +1,263 @@
+/*
+ * Licensed to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15251
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feat
Github user danix800 commented on the issue:
https://github.com/apache/spark/pull/15243
Hi, @dongjoon-hyun . Fixed!
---
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 wi
Github user danix800 commented on a diff in the pull request:
https://github.com/apache/spark/pull/15243#discussion_r80614009
--- Diff:
core/src/main/scala/org/apache/spark/deploy/worker/WorkerWatcher.scala ---
@@ -21,7 +21,7 @@ import org.apache.spark.internal.Logging
import
GitHub user danix800 opened a pull request:
https://github.com/apache/spark/pull/15251
Comment fixing
## What changes were proposed in this pull request?
Fix two comments since Actor is not used anymore.
You can merge this pull request into a Git repository by running:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15250
**[Test build #65942 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65942/consoleFull)**
for PR 15250 at commit
[`b125f2f`](https://github.com/apache/spark/commit/b
1 - 100 of 539 matches
Mail list logo