Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/12966#issuecomment-217611993
Change LGTM. Thanks @NarineK
cc @sun-rui @felixcheung @davies - any other comments on this ?
---
If your project is set up for it, you can reply to this emai
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/12974#issuecomment-217611919
@xwu0226 Because this is for Branch 1.6 only, please update the PR title to
```
[SPARK-14495][SQL] [1.6] Fix Resolution Failure of Having Clause with
Distinc
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/12974#issuecomment-217611768
@rxin @cloud-fan 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 ha
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12966#issuecomment-217611334
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12966#issuecomment-217611333
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12966#issuecomment-217611301
**[Test build #58059 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58059/consoleFull)**
for PR 12966 at commit
[`057ff9b`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12966#issuecomment-217610810
**[Test build #58059 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58059/consoleFull)**
for PR 12966 at commit
[`057ff9b`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12974#issuecomment-217609077
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 p
GitHub user xwu0226 opened a pull request:
https://github.com/apache/spark/pull/12974
[SPARK-14495][SQL] fix resolution failure of having clause with distinct
aggregate function
Symptom:
In the latest **branch 1.6**, when a `DISTINCT` aggregation function is
used in the `H
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12871#issuecomment-217608493
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12871#issuecomment-217608492
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 projec
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/10356#issuecomment-217608460
@JoshRosen Sorry, it was my mistake. It seems it works as expected and not
an issue.
```python
row = Row("f1", "f2")
>>> row(1, 2)
Row(f1=1, f2=
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12871#issuecomment-217608468
**[Test build #58055 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58055/consoleFull)**
for PR 12871 at commit
[`4d400ca`](https://g
Github user HyukjinKwon closed the pull request at:
https://github.com/apache/spark/pull/12973
---
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 i
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/12973#issuecomment-217608440
Oh I see. The usage was as below and it seems the correct behaviour
```
row = Row("f1", "f2")
>>> row(1, 2)
Row(f1=1, f2=2)
```
Sorry, cl
Github user lw-lin commented on the pull request:
https://github.com/apache/spark/pull/12725#issuecomment-217608390
I've addressed comments and expanded tests; @zsxwing would you mind taking
another look? Thanks!
---
If your project is set up for it, you can reply to this email and h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12775#issuecomment-217608341
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12775#issuecomment-217608339
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12775#issuecomment-217608311
**[Test build #58053 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58053/consoleFull)**
for PR 12775 at commit
[`ee34cd2`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12970#issuecomment-217608278
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12970#issuecomment-217608277
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12970#issuecomment-217608247
**[Test build #58052 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58052/consoleFull)**
for PR 12970 at commit
[`d1a6374`](https://g
Github user kevinyu98 commented on the pull request:
https://github.com/apache/spark/pull/12893#issuecomment-217608216
Thank you very much !
---
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 t
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12973#issuecomment-217608052
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12973#issuecomment-217608051
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12973#issuecomment-217608050
**[Test build #58058 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58058/consoleFull)**
for PR 12973 at commit
[`378f195`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12971#issuecomment-217607981
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12971#issuecomment-217607980
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12971#issuecomment-217607965
**[Test build #58056 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58056/consoleFull)**
for PR 12971 at commit
[`b717479`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12973#issuecomment-217607854
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12973#issuecomment-217607853
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12973#issuecomment-217607848
**[Test build #58057 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58057/consoleFull)**
for PR 12973 at commit
[`cf1fd05`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12973#issuecomment-217607328
**[Test build #58058 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58058/consoleFull)**
for PR 12973 at commit
[`378f195`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12725#issuecomment-217606678
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12725#issuecomment-217606677
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12725#issuecomment-217606648
**[Test build #58054 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58054/consoleFull)**
for PR 12725 at commit
[`a72423b`](https://g
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/12973#issuecomment-217606617
I submitted this PR because the author is not answering (for about four
months). If the author answers or this PR is problematic, I am happy to close
this.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12973#issuecomment-217606626
**[Test build #58057 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58057/consoleFull)**
for PR 12973 at commit
[`cf1fd05`](https://gi
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/12973#issuecomment-217606585
I submitted this PR because the author is not answering (for about four
months). If the author answers or this PR is problematic, I am happy to close
this.
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/10356#issuecomment-217606556
I submitted a PR for this in https://github.com/apache/spark/pull/10356
because I guess the author is not answering for about four months.
---
If your project is s
GitHub user HyukjinKwon opened a pull request:
https://github.com/apache/spark/pull/12973
[SPARK-12377][PySpark] Missing argments in implementation for Row.__call__
in PySpark
## What changes were proposed in this pull request?
This PR corrects the implementation of `Row.__
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/12971#discussion_r62411488
--- Diff:
mllib/src/test/java/org/apache/spark/mllib/tree/JavaDecisionTreeSuite.java ---
@@ -21,6 +21,8 @@
import java.util.HashMap;
import jav
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/12904#issuecomment-217605838
In case of writing, I think
```
Row("", "null", null)
```
should produce the CSV as below:
1. With the option, `nullValue` set to `"null
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12971#issuecomment-217605693
**[Test build #58056 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58056/consoleFull)**
for PR 12971 at commit
[`b717479`](https://gi
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12954
---
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 HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/12904#issuecomment-217605160
Here is what I think CSV datasource should handle `""`, empty string and
`nullValue`.
With the option, `nullValue` set to `"null"`, I think
```csv
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/12954#issuecomment-217605162
LGTM,
Merging this into master and 2.0 branch, thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12972#issuecomment-217605135
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12972#issuecomment-217605134
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12972#issuecomment-217605108
**[Test build #58049 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58049/consoleFull)**
for PR 12972 at commit
[`f225fe5`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12954#issuecomment-217605013
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12954#issuecomment-217605015
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/12904#issuecomment-217604986
@rxin @sureshthalamati Do you mind holding off this change until #12921 is
merged? That PR also handles `nullValue`. Apparently, I guess `nullValue` could
affect th
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/12904#discussion_r62411095
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/csv/CSVSuite.scala
---
@@ -555,4 +558,37 @@ class CSVSuite extends QueryTes
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12954#issuecomment-217604963
**[Test build #58047 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58047/consoleFull)**
for PR 12954 at commit
[`e28bbb6`](https://g
Github user sbcd90 commented on the pull request:
https://github.com/apache/spark/pull/12458#issuecomment-217604922
Hello @zsxwing ,
I have resolved the conflicts. Please have a look.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12871#issuecomment-217604019
**[Test build #58055 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58055/consoleFull)**
for PR 12871 at commit
[`4d400ca`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12971#issuecomment-217602686
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12268#issuecomment-217603358
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12921#issuecomment-217603504
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12971#issuecomment-217602678
**[Test build #58048 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58048/consoleFull)**
for PR 12971 at commit
[`b0ce8d9`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12268#issuecomment-217603319
**[Test build #58046 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58046/consoleFull)**
for PR 12268 at commit
[`f2234e3`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12921#issuecomment-217603463
**[Test build #58045 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58045/consoleFull)**
for PR 12921 at commit
[`1233bd7`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12268#issuecomment-217603359
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/12719#discussion_r62410825
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -617,6 +618,77 @@ object NullPropagation extends Rule
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/12719#discussion_r62410849
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -617,6 +618,77 @@ object NullPropagation extends Rule
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12921#issuecomment-217603505
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/10943#issuecomment-217603519
cc @rxin , looks like we missed this one...
---
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 pr
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12725#issuecomment-217601995
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12966#issuecomment-217602308
**[Test build #58050 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58050/consoleFull)**
for PR 12966 at commit
[`bf3a74d`](https://g
Github user lw-lin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12725#discussion_r62410545
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/IncrementalExecution.scala
---
@@ -27,12 +27,12 @@ import org.apache.spark.sql.exe
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12966#issuecomment-217602310
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12966#issuecomment-217602311
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12971#issuecomment-217602687
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12725#issuecomment-217601994
**[Test build #58051 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58051/consoleFull)**
for PR 12725 at commit
[`d4cd47a`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12775#issuecomment-217602287
**[Test build #58053 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58053/consoleFull)**
for PR 12775 at commit
[`ee34cd2`](https://gi
Github user lw-lin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12725#discussion_r62410548
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StreamExecution.scala
---
@@ -122,7 +122,7 @@ class StreamExecution(
* pro
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12725#issuecomment-217601996
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12725#issuecomment-217602755
**[Test build #58054 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58054/consoleFull)**
for PR 12725 at commit
[`a72423b`](https://gi
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/12893#issuecomment-217602652
thanks, merging to master and 2.0!
---
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 doe
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12970#issuecomment-217602286
**[Test build #58052 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58052/consoleFull)**
for PR 12970 at commit
[`d1a6374`](https://gi
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12893
---
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 HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/11317#issuecomment-217602140
@RussellSpitzer I saw you answered my ping before. Excuse my ping here
again.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/12953#issuecomment-217602079
thanks, merging to master and 2.0!
---
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 doe
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12953
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12725#issuecomment-217601944
**[Test build #58051 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58051/consoleFull)**
for PR 12725 at commit
[`d4cd47a`](https://gi
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/12949#discussion_r62410521
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -239,8 +239,13 @@ case class DataSource(
}
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12966#issuecomment-217601769
**[Test build #58050 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58050/consoleFull)**
for PR 12966 at commit
[`bf3a74d`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12972#issuecomment-217601777
**[Test build #58049 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58049/consoleFull)**
for PR 12972 at commit
[`f225fe5`](https://gi
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/12612#discussion_r62410489
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/metric/SQLMetrics.scala
---
@@ -19,200 +19,106 @@ package org.apache.spark.sql.execution.
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/12972#issuecomment-217601711
Let me please cc @liancheng and also @tedyu who suggested this change.
---
If your project is set up for it, you can reply to this email and have your
reply appear
GitHub user HyukjinKwon opened a pull request:
https://github.com/apache/spark/pull/12972
[SPARK-15198][SQL] Support for pushing down filters for boolean types in
ORC data source
## What changes were proposed in this pull request?
This PR adds the support for pushing filter
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/12899#discussion_r62410420
--- Diff: core/src/main/scala/org/apache/spark/scheduler/Task.scala ---
@@ -155,7 +155,13 @@ private[spark] abstract class Task[T](
*/
def co
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12971#issuecomment-217601538
**[Test build #58048 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58048/consoleFull)**
for PR 12971 at commit
[`b0ce8d9`](https://gi
GitHub user hhbyyh opened a pull request:
https://github.com/apache/spark/pull/12971
[SPARK-14814][MLlib] API: Java compatibility, docs
## What changes were proposed in this pull request?
fix a java compatibility function in mllib DecisionTreeModel
## How was this p
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12954#issuecomment-217600276
**[Test build #58047 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58047/consoleFull)**
for PR 12954 at commit
[`e28bbb6`](https://gi
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12113
---
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 davies commented on the pull request:
https://github.com/apache/spark/pull/12113#issuecomment-21767
Merging this into master and 2.0 branch, thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/12954#discussion_r62409983
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/BooleanSimplification.scala
---
@@ -0,0 +1,124 @@
+/*
+ * Licensed to th
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/12954#discussion_r62409870
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/BooleanSimplification.scala
---
@@ -0,0 +1,124 @@
+/*
+ * Licensed t
1 - 100 of 657 matches
Mail list logo