Github user lresende commented on the issue:
https://github.com/apache/spark/pull/14601
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 wishe
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13373
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63612/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13373
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
enabled
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13373
**[Test build #63612 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63612/consoleFull)**
for PR 13373 at commit
[`c7b181e`](https://github.com/apache/spark/commit/
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14568#discussion_r74437337
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/MonotonicallyIncreasingID.scala
---
@@ -40,13 +42,23 @@ import org.apache
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13373
**[Test build #63612 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63612/consoleFull)**
for PR 13373 at commit
[`c7b181e`](https://github.com/apache/spark/commit/c
Github user nchammas commented on the issue:
https://github.com/apache/spark/pull/14579
> So there is no chaining requirement, and it will only work in a with
statement.
@MLnick - Couldn't we also create a scenario (like @holdenk did earlier)
where a user does something like
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14597#discussion_r74437070
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/feature/ChiSqSelector.scala ---
@@ -197,3 +197,28 @@ class ChiSqSelector @Since("1.3.0") (
new
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14568#discussion_r74436746
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/MonotonicallyIncreasingID.scala
---
@@ -81,3 +93,12 @@ case class Monoton
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14568#discussion_r74434465
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/MonotonicallyIncreasingID.scala
---
@@ -40,13 +42,23 @@ import org.apache
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14599
The `L`, `S` & `Y` suffixes come from Hive.
The fix that is proposed by @petermaxlee has a potential problem when we
try parse something like `a-1`. This will be tokenized into an `IDENTI
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14568
**[Test build #63611 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63611/consoleFull)**
for PR 14568 at commit
[`e547459`](https://github.com/apache/spark/commit/e
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14548
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 fe
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14548#discussion_r74427941
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/subquery.scala ---
@@ -17,42 +17,77 @@
package org.apache.spark.sql.execution
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14598#discussion_r74427705
--- Diff: sql/core/src/test/resources/sql-tests/results/literals.sql.out ---
@@ -0,0 +1,148 @@
+-- Automatically generated by SQLQueryTestSuite
+--
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/14321#discussion_r74426713
--- Diff:
core/src/test/scala/org/apache/spark/rdd/PairRDDFunctionsSuite.scala ---
@@ -168,6 +168,126 @@ class PairRDDFunctionsSuite extends SparkFunSuite
wi
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/14321#discussion_r74426625
--- Diff:
core/src/test/scala/org/apache/spark/rdd/PairRDDFunctionsSuite.scala ---
@@ -168,6 +168,126 @@ class PairRDDFunctionsSuite extends SparkFunSuite
wi
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14598#discussion_r74426121
--- Diff: sql/core/src/test/resources/sql-tests/results/literals.sql.out ---
@@ -0,0 +1,148 @@
+-- Automatically generated by SQLQueryTestSuite
+--
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14598
This is pretty cool :)
I am comparing this to the `ExpressionParserSuite`. Shouldn't we add
support for more complex string cases, intervals and a type constructors, see:
https://github.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14602
**[Test build #63610 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63610/consoleFull)**
for PR 14602 at commit
[`e3e8929`](https://github.com/apache/spark/commit/e
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/14321#discussion_r74425472
--- Diff:
core/src/test/scala/org/apache/spark/rdd/PairRDDFunctionsSuite.scala ---
@@ -662,6 +796,67 @@ class PairRDDFunctionsSuite extends SparkFunSuite with
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14598#discussion_r74424129
--- Diff: sql/core/src/test/resources/sql-tests/inputs/literals.sql ---
@@ -0,0 +1,52 @@
+-- Literal parsing
+
+-- null
+select null;
+
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14599#discussion_r74423461
--- Diff:
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
@@ -627,6 +627,7 @@ quotedIdentifier
number
: DEC
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14530
As to the actual logic of the change, I don't feel that confident about
reviewing it. Suggest looking at 'blame' output here to see who might have
written it?
---
If your project is set up for it,
Github user SaintBacchus commented on the issue:
https://github.com/apache/spark/pull/14530
\cc @srowen
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, o
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14603
**[Test build #63609 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63609/consoleFull)**
for PR 14603 at commit
[`ba1bae0`](https://github.com/apache/spark/commit/b
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14603
cc @thunterdb @yhuai
---
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 opened a pull request:
https://github.com/apache/spark/pull/14603
[SPARK-17021][SQL] simplify the constructor parameters of QuantileSummaries
## What changes were proposed in this pull request?
1. `sampled` doesn't need to be `ArrayBuffer`, we never upd
Github user mpjlu commented on a diff in the pull request:
https://github.com/apache/spark/pull/14597#discussion_r74416260
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/feature/ChiSqSelector.scala ---
@@ -197,3 +197,28 @@ class ChiSqSelector @Since("1.3.0") (
new C
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14597#discussion_r74413499
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/feature/ChiSqSelector.scala ---
@@ -197,3 +197,28 @@ class ChiSqSelector @Since("1.3.0") (
new
Github user mpjlu commented on a diff in the pull request:
https://github.com/apache/spark/pull/14597#discussion_r74412725
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/feature/ChiSqSelector.scala ---
@@ -197,3 +197,28 @@ class ChiSqSelector @Since("1.3.0") (
new C
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14102
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63605/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14602
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/14602
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63607/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14602
**[Test build #63607 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63607/consoleFull)**
for PR 14602 at commit
[`94570ae`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14102
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/14102
**[Test build #63605 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63605/consoleFull)**
for PR 14102 at commit
[`80b2d3a`](https://github.com/apache/spark/commit/
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14600
Thanks @avulanov for the follow up. Merging to 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 does not have this f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14600
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/14600
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63604/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14600
**[Test build #63604 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63604/consoleFull)**
for PR 14600 at commit
[`2f4566c`](https://github.com/apache/spark/commit/
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/12896
@srowen @jkbradley any further comments / issues? I plan to create
follow-up JIRAs for docs & examples, as well as expansions to the cold-start
strategies.
also cc @mengxr @yanboliang for an
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14559
+1 for both OK and not exposing `JDBCOptions`. I know it is slightly
different but `CSVoptions` and `JSONOptions` would be good references to follow.
---
If your project is set up for it, you c
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14548
Cool picture!
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/14593
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 featur
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14597#discussion_r74404978
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/feature/ChiSqSelector.scala ---
@@ -197,3 +197,28 @@ class ChiSqSelector @Since("1.3.0") (
new
Github user mpjlu commented on a diff in the pull request:
https://github.com/apache/spark/pull/14597#discussion_r74404348
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/feature/ChiSqSelector.scala ---
@@ -197,3 +197,28 @@ class ChiSqSelector @Since("1.3.0") (
new C
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14593
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63608/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14593
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/14593
**[Test build #63608 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63608/consoleFull)**
for PR 14593 at commit
[`c438d1e`](https://github.com/apache/spark/commit/
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14559
This is looking good. I'm not as familiar with the Python API but it
doesn't seem like this needs a change in an API anywhere, Scala or Python. So I
think this is OK.
---
If your project is set up
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14598#discussion_r74403321
--- Diff: sql/core/src/test/resources/sql-tests/results/literals.sql.out ---
@@ -0,0 +1,136 @@
+-- Automatically generated by SQLQueryTestSuite
+--
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14598#discussion_r74403147
--- Diff: sql/core/src/test/resources/sql-tests/results/literals.sql.out ---
@@ -0,0 +1,136 @@
+-- Automatically generated by SQLQueryTestSuite
+--
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14593
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63606/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14593
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/14593
**[Test build #63606 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63606/consoleFull)**
for PR 14593 at commit
[`9d899dc`](https://github.com/apache/spark/commit/
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14598#discussion_r74402656
--- Diff: sql/core/src/test/resources/sql-tests/results/literals.sql.out ---
@@ -0,0 +1,136 @@
+-- Automatically generated by SQLQueryTestSuite
+--
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14593
**[Test build #63608 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63608/consoleFull)**
for PR 14593 at commit
[`c438d1e`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14102
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63602/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14102
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/14102
**[Test build #63602 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63602/consoleFull)**
for PR 14102 at commit
[`a8b6a04`](https://github.com/apache/spark/commit/
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/14593#discussion_r74401530
--- Diff: docs/graphx-programming-guide.md ---
@@ -235,8 +235,9 @@ triplet view of a graph to render a collection of
strings describing relationshi
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14602
**[Test build #63607 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63607/consoleFull)**
for PR 14602 at commit
[`94570ae`](https://github.com/apache/spark/commit/9
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14593#discussion_r74401289
--- Diff: docs/graphx-programming-guide.md ---
@@ -235,8 +235,9 @@ triplet view of a graph to render a collection of
strings describing relationshi
val g
GitHub user srowen opened a pull request:
https://github.com/apache/spark/pull/14602
[SPARK-16966] [SQL] [CORE] App Name is a randomUUID even when
"spark.app.name" exists
## What changes were proposed in this pull request?
Don't override spark.app.name in SparkSession when
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14601
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 HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/14593#discussion_r74400900
--- Diff: docs/streaming-programming-guide.md ---
@@ -1142,12 +1142,12 @@ val joinedStream = windowedStream.transform { rdd
=> rdd.join(dataset) }
J
GitHub user agsachin opened a pull request:
https://github.com/apache/spark/pull/14601
[SPARK-13979][Spark Core][WIP]Killed executor is re spawned without AWS
keyâ¦
## What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
*Summe
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14552
---
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 srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14597#discussion_r74400488
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/feature/ChiSqSelector.scala ---
@@ -197,3 +197,28 @@ class ChiSqSelector @Since("1.3.0") (
new
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14593
**[Test build #63606 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63606/consoleFull)**
for PR 14593 at commit
[`9d899dc`](https://github.com/apache/spark/commit/9
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14552
Merged to master
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or i
Github user mpjlu commented on a diff in the pull request:
https://github.com/apache/spark/pull/14597#discussion_r74400235
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/feature/ChiSqSelector.scala ---
@@ -197,3 +197,28 @@ class ChiSqSelector @Since("1.3.0") (
new C
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14102
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63599/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14102
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/14564
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/14524
@mengxr do you happen to have an opinion on this?
---
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/14491
---
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 issue:
https://github.com/apache/spark/pull/14102
**[Test build #63599 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63599/consoleFull)**
for PR 14102 at commit
[`aaac7db`](https://github.com/apache/spark/commit/
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14564
Merged to master
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or i
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14491
Let's close this in favor of https://github.com/apache/spark/pull/14564
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/14534
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/14534
Merged to master
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or i
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14581
---
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 clockfly commented on the issue:
https://github.com/apache/spark/pull/14546
I believe this doesn't fix all the cases.
How about
```
sql("select count(*), a from (select 1 as a) tmp group by 2 having a >
0").show
```
---
If your project is set up for i
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14581
merged to master/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 does not have this feature
enabled and wishes so,
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14102
Thanks for reviewing this!
---
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/14102
**[Test build #63605 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63605/consoleFull)**
for PR 14102 at commit
[`80b2d3a`](https://github.com/apache/spark/commit/8
Github user notflorian commented on the issue:
https://github.com/apache/spark/pull/12285
Sorry to camp this old issue.
With spark 1.6.1, I've a lot of executor OOM when I do a
rdd.saveAsHadoopFile, with parameter classOf[GzipCodec], to gzip the outpout
files.
Witho
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14598
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63597/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14598
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/14397
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63595/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14598
**[Test build #63597 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63597/consoleFull)**
for PR 14598 at commit
[`243cd39`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14397
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 mpjlu commented on the issue:
https://github.com/apache/spark/pull/14597
Hi, @srowen , I can modify the implementation in .ml to accommodate the new
params. Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as w
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14597#discussion_r74397653
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/feature/ChiSqSelector.scala ---
@@ -197,3 +197,28 @@ class ChiSqSelector @Since("1.3.0") (
new
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14397
**[Test build #63595 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63595/consoleFull)**
for PR 14397 at commit
[`7f74ec7`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14600
**[Test build #63604 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63604/consoleFull)**
for PR 14600 at commit
[`2f4566c`](https://github.com/apache/spark/commit/2
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14593
@srowen Thank you for review! I can revert non-visible ones but will just
leave the visible ones.
I am okay with reverting this (anyway now we know there are nits of spacing
in documen
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14600
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
301 - 400 of 527 matches
Mail list logo