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) }
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
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 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
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
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 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
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 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)
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
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 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 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
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
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
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
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
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
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
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
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
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
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
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/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
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 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 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 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
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
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
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
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
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
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 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
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
:
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 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
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
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 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
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
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
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 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
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 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
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
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
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 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,
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 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
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
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
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
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
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
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
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
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 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
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 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 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
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14568
**[Test build #63616 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63616/consoleFull)**
for PR 14568 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14586
**[Test build #3219 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3219/consoleFull)**
for PR 14586 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14586
**[Test build #3218 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3218/consoleFull)**
for PR 14586 at commit
Github user nchammas commented on the issue:
https://github.com/apache/spark/pull/14579
Hmm, OK I see. (Apologies, I don't understand what pipelined RDDs are for,
so the examples are going a bit over my head. ð
)
---
If your project is set up for it, you can reply to this email
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
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 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
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 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
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 lresende commented on a diff in the pull request:
https://github.com/apache/spark/pull/14601#discussion_r74440796
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkHadoopUtil.scala
---
@@ -107,6 +107,14 @@ class SparkHadoopUtil extends Logging {
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14568#discussion_r74442456
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/MonotonicallyIncreasingID.scala
---
@@ -40,13 +42,23 @@ import
GitHub user WeichenXu123 opened a pull request:
https://github.com/apache/spark/pull/14604
[Doc] add config option spark.ui.enabled into document
## What changes were proposed in this pull request?
The configuration doc lost the config option `spark.ui.enabled` (default
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14604#discussion_r7575
--- Diff: docs/configuration.md ---
@@ -568,6 +568,13 @@ Apart from these, the following properties are also
available, and may be useful
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 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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14604
**[Test build #63615 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63615/consoleFull)**
for PR 14604 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14604
**[Test build #63615 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63615/consoleFull)**
for PR 14604 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14604
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/14568
**[Test build #63611 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63611/consoleFull)**
for PR 14568 at commit
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/14579
@nchammas to be clear - subclassing only breaks pipelining if the
persisted_rdd is later unpersisted (e.g. used with a `with` statement or
otherwise) - otherwise you can't pipeline on top of a
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:
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
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14603
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63609/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14568
**[Test build #63613 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63613/consoleFull)**
for PR 14568 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14456
Oh dear. I give up on back-porting things to 1.6. It just breaks too much
at this stage! I will revert.
---
If your project is set up for it, you can reply to this email and have your
reply appear
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 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
1 - 100 of 558 matches
Mail list logo