Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11930#issuecomment-200698788
**[Test build #54017 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54017/consoleFull)**
for PR 11930 at commit
[`aea6146`](https://gi
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11929#discussion_r57280163
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/AbstractSparkSQLParser.scala
---
@@ -129,7 +129,7 @@ class SqlLexical extends StdLexi
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/11930
[SPARK-14112] [SQL] [WIP] Unique Constraints over a Set of
AttributeReferences
What changes were proposed in this pull request?
This PR is to introduce unique constraints over a set of
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11916#issuecomment-200698229
**[Test build #54016 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54016/consoleFull)**
for PR 11916 at commit
[`7bbe734`](https://gi
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11557#discussion_r57279846
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/parser/ng/PlanParserSuite.scala
---
@@ -0,0 +1,408 @@
+/*
+ * Licensed to the
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11926#issuecomment-200697329
**[Test build #54015 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54015/consoleFull)**
for PR 11926 at commit
[`5912fd3`](https://gi
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11557#discussion_r57279658
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/parser/ng/PlanParserSuite.scala
---
@@ -0,0 +1,408 @@
+/*
+ * Licensed to the
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11557#discussion_r57279657
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/parser/ng/PlanParserSuite.scala
---
@@ -0,0 +1,408 @@
+/*
+ * Licensed to the
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11578#issuecomment-200697073
me 2.
---
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
enab
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/11926#issuecomment-200696828
retest this please.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this fe
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11578#issuecomment-200696780
**[Test build #54014 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54014/consoleFull)**
for PR 11578 at commit
[`d51d553`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11854#issuecomment-200696769
**[Test build #54013 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54013/consoleFull)**
for PR 11854 at commit
[`7d95bc1`](https://gi
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/11578#issuecomment-200696715
My another PR also gets this error...
---
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 pull request:
https://github.com/apache/spark/pull/11885#issuecomment-200696725
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/11885#issuecomment-200696706
**[Test build #54011 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54011/consoleFull)**
for PR 11885 at commit
[`6702927`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11885#issuecomment-200696727
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 hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11557#discussion_r57279440
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/parser/ng/ExpressionParserSuite.scala
---
@@ -0,0 +1,494 @@
+/*
+ * Licensed
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11557#discussion_r57279420
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/parser/ng/ExpressionParserSuite.scala
---
@@ -0,0 +1,494 @@
+/*
+ * Licensed
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/11557#discussion_r57279436
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/parser/ng/PlanParserSuite.scala
---
@@ -0,0 +1,408 @@
+/*
+ * Licensed to the
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11578#issuecomment-200696574
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11854#issuecomment-200696278
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11578#issuecomment-200696245
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 hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11557#discussion_r57279310
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/parser/ng/ExpressionParserSuite.scala
---
@@ -0,0 +1,494 @@
+/*
+ * Licensed
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11578#issuecomment-200696247
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/11578#issuecomment-200696143
**[Test build #54001 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54001/consoleFull)**
for PR 11578 at commit
[`d51d553`](https://g
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/11891#issuecomment-200696198
Ah, unfortunately we just reverted #11836 because it caused some
regressions. This caused another compilation error for your PR... Sorry for the
trouble...
---
If y
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/11557#discussion_r57279151
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/parser/ng/PlanParserSuite.scala
---
@@ -0,0 +1,408 @@
+/*
+ * Licensed to the
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11557#discussion_r57279097
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/parser/ng/ExpressionParserSuite.scala
---
@@ -0,0 +1,494 @@
+/*
+ * Licensed
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11557#discussion_r57279041
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -0,0 +1,250 @@
+/*
+ * Licensed to the Apache Softwar
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/11557#discussion_r57279004
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/parser/ng/PlanParserSuite.scala
---
@@ -0,0 +1,408 @@
+/*
+ * Licensed to the
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11557#discussion_r57278951
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/parser/ng/PlanParserSuite.scala
---
@@ -0,0 +1,408 @@
+/*
+ * Licensed to the
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/11557#discussion_r57278901
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/parser/ng/ExpressionParserSuite.scala
---
@@ -0,0 +1,494 @@
+/*
+ * Licensed t
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/11557#discussion_r57278832
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/parser/ng/PlanParserSuite.scala
---
@@ -0,0 +1,408 @@
+/*
+ * Licensed to the
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11809#issuecomment-200694455
**[Test build #54012 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54012/consoleFull)**
for PR 11809 at commit
[`8e8dd72`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11885#issuecomment-200694456
**[Test build #54011 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54011/consoleFull)**
for PR 11885 at commit
[`6702927`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11891#issuecomment-200694397
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/11891#issuecomment-200694386
**[Test build #54010 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54010/consoleFull)**
for PR 11891 at commit
[`c4111fa`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11891#issuecomment-200694394
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/11926#issuecomment-200694330
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/11926#issuecomment-200694329
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/11926#issuecomment-200694233
**[Test build #54002 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54002/consoleFull)**
for PR 11926 at commit
[`5912fd3`](https://g
Github user wangyang1992 commented on the pull request:
https://github.com/apache/spark/pull/11929#issuecomment-200694207
BTW, I cannot reproduce this problem in master. I pushed this pr in case
there will be another release in this branch.
---
If your project is set up for it, you c
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11891#issuecomment-200693807
**[Test build #54010 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54010/consoleFull)**
for PR 11891 at commit
[`c4111fa`](https://gi
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/11891#issuecomment-200693267
add to whitelist
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this fe
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11929#discussion_r57278328
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/AbstractSparkSQLParser.scala
---
@@ -129,7 +129,7 @@ class SqlLexical extends StdLexical {
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/11891#issuecomment-200693244
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/11891#discussion_r57278305
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/orc/OrcQuerySuite.scala ---
@@ -400,4 +400,33 @@ class OrcQuerySuite extends QueryTest with
Be
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11854#issuecomment-200693195
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 rxin commented on the pull request:
https://github.com/apache/spark/pull/11723#issuecomment-200693181
Is this something Facebook needs too?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11854#issuecomment-200693196
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/11854#issuecomment-200693098
**[Test build #54003 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54003/consoleFull)**
for PR 11854 at commit
[`7d95bc1`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11916#issuecomment-200690718
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/11916#issuecomment-200690724
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/11916#issuecomment-200690703
**[Test build #54009 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54009/consoleFull)**
for PR 11916 at commit
[`53023dd`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11916#issuecomment-200690203
**[Test build #54009 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54009/consoleFull)**
for PR 11916 at commit
[`53023dd`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11895#issuecomment-200687758
**[Test build #54008 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54008/consoleFull)**
for PR 11895 at commit
[`f5d5976`](https://gi
Github user tejasapatil commented on the pull request:
https://github.com/apache/spark/pull/11723#issuecomment-200687233
@rxin : I would really like to have this PR in trunk. As things stand, for
anyone using their own scheduler, one has to maintain a patch over open source
release to
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/11895#discussion_r57277319
--- Diff:
sql/core/src/test/java/test/org/apache/spark/sql/JavaDataFrameSuite.java ---
@@ -303,27 +305,42 @@ public void pivot() {
Assert.assertEqua
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/11895#discussion_r57277180
--- Diff:
sql/core/src/test/java/test/org/apache/spark/sql/JavaDataFrameSuite.java ---
@@ -303,27 +305,42 @@ public void pivot() {
Assert.assertEqua
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11895#issuecomment-200684929
**[Test build #54007 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54007/consoleFull)**
for PR 11895 at commit
[`14d859d`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11925#issuecomment-200682795
**[Test build #54006 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54006/consoleFull)**
for PR 11925 at commit
[`dcf8096`](https://gi
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/11723#discussion_r57276779
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/ExternalClusterManagerSuite.scala
---
@@ -0,0 +1,65 @@
+/*
+ * Licensed to the Apache S
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/11723#discussion_r57276711
--- Diff: dev/.rat-excludes ---
@@ -98,3 +98,4 @@ LZ4BlockInputStream.java
spark-deps-.*
.*csv
.*tsv
+org.apache.spark.scheduler.Extern
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/11723#discussion_r57276691
--- Diff:
core/src/test/resources/META-INF/services/org.apache.spark.scheduler.ExternalClusterManager
---
@@ -0,0 +1 @@
+org.apache.spark.scheduler.
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/11723#discussion_r57276663
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/ExternalClusterManager.scala ---
@@ -0,0 +1,62 @@
+/*
+ * Licensed to the Apache Softwar
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/11723#discussion_r57276660
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/ExternalClusterManager.scala ---
@@ -0,0 +1,62 @@
+/*
+ * Licensed to the Apache Softwar
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/11723#discussion_r57276618
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/ExternalClusterManagerSuite.scala
---
@@ -0,0 +1,65 @@
+/*
+ * Licensed to the Apache S
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11916#issuecomment-200679647
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/11916#issuecomment-200679645
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/11922#issuecomment-200679591
**[Test build #54005 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54005/consoleFull)**
for PR 11922 at commit
[`54bd806`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11916#issuecomment-200679249
**[Test build #53998 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53998/consoleFull)**
for PR 11916 at commit
[`7c033b6`](https://g
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/11723#discussion_r57276344
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/ExternalClusterManager.scala ---
@@ -0,0 +1,62 @@
+/*
+ * Licensed to the Apache Softwar
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11809#issuecomment-200678468
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 tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/11723#discussion_r57276268
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -149,7 +149,14 @@ private[spark] class Executor(
tr.kill(interrup
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11809#issuecomment-200678465
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/11809#issuecomment-200677948
**[Test build #53999 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53999/consoleFull)**
for PR 11809 at commit
[`81c46c7`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11824#issuecomment-200677903
**[Test build #54004 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54004/consoleFull)**
for PR 11824 at commit
[`dcdd3cb`](https://gi
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/11723#discussion_r57276201
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -149,7 +149,14 @@ private[spark] class Executor(
tr.kill(interrup
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/11723#discussion_r57276211
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -149,7 +149,14 @@ private[spark] class Executor(
tr.kill(interrup
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/11824#issuecomment-200676764
Retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this fe
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11854#issuecomment-200676690
The motivation of this function `distinctSet` is to obtain the uniqueness
constraint from the child operators. The output of `Distinct`, `Intersect`,
`Except`, and `
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/11723#discussion_r57276135
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -2443,8 +2443,34 @@ object SparkContext extends Logging {
"in the
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11928#issuecomment-200676208
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/11928#issuecomment-200676206
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/11928#issuecomment-200675840
**[Test build #54000 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54000/consoleFull)**
for PR 11928 at commit
[`836ec92`](https://g
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11927#issuecomment-200674840
LGTM pending Jenkins.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11927#issuecomment-200675269
**[Test build #2676 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2676/consoleFull)**
for PR 11927 at commit
[`9929da9`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11929#issuecomment-200674607
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 wangyang1992 opened a pull request:
https://github.com/apache/spark/pull/11929
[SPARK-13934][SQL] fixed table identifier
## What changes were proposed in this pull request?
Table identifier that starts in a form of scientific notation (like 1e34)
will throw an except
Github user andrewor14 closed the pull request at:
https://github.com/apache/spark/pull/11923
---
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 andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/11923#issuecomment-200672391
I'm going to close this for now since we reverted the original patch.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/11927#discussion_r57275794
--- Diff: core/src/main/scala/org/apache/spark/rdd/PipedRDD.scala ---
@@ -205,6 +206,8 @@ private[spark] class PipedRDD[T: ClassTag](
private d
Github user yinxusen commented on the pull request:
https://github.com/apache/spark/pull/9#issuecomment-200672196
@jkbradley Sure, I'll leave it here and check back later. :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11927#discussion_r57275342
--- Diff: core/src/main/scala/org/apache/spark/rdd/PipedRDD.scala ---
@@ -205,6 +206,8 @@ private[spark] class PipedRDD[T: ClassTag](
private def prop
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/11621#discussion_r57275251
--- Diff: python/pyspark/ml/wrapper.py ---
@@ -223,3 +223,20 @@ def _call_java(self, name, *args):
sc = SparkContext._active_spark_context
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11854#issuecomment-200665778
**[Test build #54003 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54003/consoleFull)**
for PR 11854 at commit
[`7d95bc1`](https://gi
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11854#issuecomment-200665098
Added a function `distinctSet` into `QueryPlan`. This function will return
the set of attributes whose combination can uniquely identify a row. Maybe I
should create
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11926#issuecomment-200663369
**[Test build #54002 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54002/consoleFull)**
for PR 11926 at commit
[`5912fd3`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11926#issuecomment-200663292
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 viirya commented on the pull request:
https://github.com/apache/spark/pull/11926#issuecomment-200663281
retest this please.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this fe
1 - 100 of 874 matches
Mail list logo