Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11485#issuecomment-193791247
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/11485#issuecomment-193791254
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/11530#issuecomment-193791292
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/11530#issuecomment-193790965
**[Test build #52662 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52662/consoleFull)**
for PR 11530 at commit
[`cdb6077`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11485#issuecomment-193790898
**[Test build #52666 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52666/consoleFull)**
for PR 11485 at commit
[`4f31c5c`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11443#issuecomment-193780100
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/11443#issuecomment-193780096
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/11443#issuecomment-193779928
**[Test build #52656 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52656/consoleFull)**
for PR 11443 at commit
[`42e7441`](https://g
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11560#discussion_r55356255
--- Diff:
mllib/src/test/java/org/apache/spark/mllib/recommendation/JavaALSSuite.java ---
@@ -48,14 +47,14 @@ public void tearDown() {
sc = null;
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11523#issuecomment-193779451
OK, I'd like to merge this one too, though I'm a little gunshy about
changing the build structure today. I'll let it sit a moment.
---
If your project is set up for it,
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11523#issuecomment-193777097
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/11523#issuecomment-193777094
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/11523#issuecomment-193776845
**[Test build #52659 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52659/consoleFull)**
for PR 11523 at commit
[`f299bb6`](https://g
Github user WangTaoTheTonic commented on the pull request:
https://github.com/apache/spark/pull/11358#issuecomment-193776099
hi @vanzin, how about spark sql in this issue, in your view? as in spark
sql it will revoke SessionState.start in which will finally connected to
metastore in H
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11538#issuecomment-193771026
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/11538#issuecomment-193771034
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11538#issuecomment-193770318
**[Test build #52660 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52660/consoleFull)**
for PR 11538 at commit
[`ba6411a`](https://g
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11573#discussion_r55352952
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkQl.scala ---
@@ -64,6 +85,53 @@ private[sql] class SparkQl(conf: ParserConf =
Simpl
Github user zjffdu commented on the pull request:
https://github.com/apache/spark/pull/11211#issuecomment-193763584
Ping @JoshRosen
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this featu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11443#issuecomment-193759835
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/11443#issuecomment-193759841
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/11443#issuecomment-193759455
**[Test build #52658 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52658/consoleFull)**
for PR 11443 at commit
[`50be32b`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11485#issuecomment-193758948
**[Test build #52666 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52666/consoleFull)**
for PR 11485 at commit
[`4f31c5c`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11443#issuecomment-193758805
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/11443#issuecomment-193758802
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/11443#issuecomment-193758278
**[Test build #52657 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52657/consoleFull)**
for PR 11443 at commit
[`c90bd96`](https://g
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/11485#issuecomment-193757000
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11555#issuecomment-193756848
**[Test build #52665 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52665/consoleFull)**
for PR 11555 at commit
[`054f50a`](https://gi
Github user pravingadakh commented on a diff in the pull request:
https://github.com/apache/spark/pull/11575#discussion_r55349423
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/feature/StringIndexerSuite.scala ---
@@ -178,6 +178,20 @@ class StringIndexerSuite
}
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11555#issuecomment-193750275
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/11555#issuecomment-193750264
**[Test build #52664 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52664/consoleFull)**
for PR 11555 at commit
[`fcd60de`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11555#issuecomment-193750272
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/11555#issuecomment-193749839
**[Test build #52664 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52664/consoleFull)**
for PR 11555 at commit
[`fcd60de`](https://gi
Github user dongjoon-hyun commented on the pull request:
https://github.com/apache/spark/pull/11530#issuecomment-193746151
Hi, @srowen . I've finished.
Let's see the result, [Jenkins
#52663](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52663/consoleFull)
--
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11530#issuecomment-193745485
**[Test build #52663 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52663/consoleFull)**
for PR 11530 at commit
[`e3c4cf0`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11410#issuecomment-193744957
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/11410#issuecomment-193744962
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11410#issuecomment-193744207
**[Test build #52650 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52650/consoleFull)**
for PR 11410 at commit
[`8b79215`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11547#issuecomment-193740426
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/11547#issuecomment-193740423
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/11547#issuecomment-193740105
**[Test build #52661 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52661/consoleFull)**
for PR 11547 at commit
[`4a92dfd`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11530#issuecomment-193740232
**[Test build #52662 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52662/consoleFull)**
for PR 11530 at commit
[`cdb6077`](https://gi
Github user jaceklaskowski commented on a diff in the pull request:
https://github.com/apache/spark/pull/10428#discussion_r55347052
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala
---
@@ -126,6 +144,13 @@ case class Hour(ch
Github user MLnick commented on the pull request:
https://github.com/apache/spark/pull/11547#issuecomment-193732035
ok to test
---
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
ena
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11547#issuecomment-193733671
**[Test build #52661 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52661/consoleFull)**
for PR 11547 at commit
[`4a92dfd`](https://gi
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/11530#discussion_r55345620
--- Diff:
examples/src/main/java/org/apache/spark/examples/streaming/JavaTwitterHashTagJoinSentiments.java
---
@@ -171,5 +171,6 @@ public void call(Ja
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/11576#issuecomment-193731589
@yhuai Could you check 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 ha
Github user dongjoon-hyun commented on the pull request:
https://github.com/apache/spark/pull/11567#issuecomment-193731456
Thank you for merging.
---
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 jaceklaskowski commented on a diff in the pull request:
https://github.com/apache/spark/pull/11575#discussion_r55345145
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/feature/StringIndexerSuite.scala ---
@@ -178,6 +178,20 @@ class StringIndexerSuite
}
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11538#issuecomment-193730128
**[Test build #52660 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52660/consoleFull)**
for PR 11538 at commit
[`ba6411a`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11576#issuecomment-193729884
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/11576#issuecomment-193729881
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/11576#issuecomment-193729610
**[Test build #52652 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52652/consoleFull)**
for PR 11576 at commit
[`6332f94`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11555#issuecomment-193725131
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/11555#issuecomment-193725127
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/11555#issuecomment-193724658
**[Test build #52649 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52649/consoleFull)**
for PR 11555 at commit
[`1ebb3c5`](https://g
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11530#discussion_r55343410
--- Diff:
examples/src/main/java/org/apache/spark/examples/streaming/JavaTwitterHashTagJoinSentiments.java
---
@@ -171,5 +171,6 @@ public void call(JavaPairR
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11490
---
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 pull request:
https://github.com/apache/spark/pull/11490#issuecomment-193722345
OK. 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
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11487#issuecomment-193720957
OK, that seems reasonable. I was more wondering if we risk some deadlock in
2.11, but the tests are also potentially testing for that problem as well.
---
If your proje
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/11235#issuecomment-193720918
Comments addressed, please check if the change is good to merge now. Thanks!
---
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/11523#issuecomment-193718351
**[Test build #52659 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52659/consoleFull)**
for PR 11523 at commit
[`f299bb6`](https://gi
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11523#issuecomment-193716967
Jenkins, 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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11443#issuecomment-193711944
**[Test build #52658 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52658/consoleFull)**
for PR 11443 at commit
[`50be32b`](https://gi
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11567
---
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/11572#issuecomment-193709857
While creating a PR I also saw some nits.
At 'OrcRelation',
1. [`import
com.google.common.base.Objects`](https://github.com/apache/spark/blob/mast
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11567#issuecomment-193709656
Merged to master. Thanks all, and sorry about that. I am not sure why the
PR builder passed.
---
If your project is set up for it, you can reply to this email and have
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11443#issuecomment-193709463
**[Test build #52657 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52657/consoleFull)**
for PR 11443 at commit
[`c90bd96`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11567#issuecomment-193708863
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/11567#issuecomment-193708860
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/11567#issuecomment-193708216
**[Test build #52648 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52648/consoleFull)**
for PR 11567 at commit
[`fbac4a1`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11577#issuecomment-193708276
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/11538#issuecomment-193708141
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/11538#issuecomment-193708134
**[Test build #52655 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52655/consoleFull)**
for PR 11538 at commit
[`07af901`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11577#issuecomment-193708272
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/11538#issuecomment-193708146
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/11577#issuecomment-193707958
**[Test build #52654 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52654/consoleFull)**
for PR 11577 at commit
[`e30160e`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11538#issuecomment-193707287
**[Test build #52655 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52655/consoleFull)**
for PR 11538 at commit
[`07af901`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11443#issuecomment-193707276
**[Test build #52656 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52656/consoleFull)**
for PR 11443 at commit
[`42e7441`](https://gi
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/11572#discussion_r55340181
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -34,14 +34,39 @@ import org.apache.spark.sql.sources.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11557#issuecomment-193704911
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/11557#issuecomment-193704913
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11557#issuecomment-193704464
**[Test build #52646 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52646/consoleFull)**
for PR 11557 at commit
[`2c9a092`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11577#issuecomment-193701440
**[Test build #52654 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52654/consoleFull)**
for PR 11577 at commit
[`e30160e`](https://gi
Github user jerryshao commented on the pull request:
https://github.com/apache/spark/pull/11577#issuecomment-193700757
CC @vanzin .
---
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 featur
GitHub user jerryshao opened a pull request:
https://github.com/apache/spark/pull/11577
[HOTFIX][Yarn] Fix yarn cluster mode fire and forget regression
## What changes were proposed in this pull request?
Fire and forget is disabled by default, with this patch #10205 it is
e
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11573#discussion_r55337949
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala ---
@@ -0,0 +1,197 @@
+/*
+ * Licensed to the Apache Software F
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11573#discussion_r55337179
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/bucket.scala
---
@@ -29,7 +32,8 @@ package org.apache.spark.sql.execution.da
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11573#discussion_r55336977
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AlterTableCommandParser.scala
---
@@ -0,0 +1,410 @@
+/*
+ * Licensed to
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11573#discussion_r55336439
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AlterTableCommandParser.scala
---
@@ -0,0 +1,410 @@
+/*
+ * Licensed to
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11443#issuecomment-193690192
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/11443#issuecomment-193690169
**[Test build #52653 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52653/consoleFull)**
for PR 11443 at commit
[`46e3340`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11443#issuecomment-193690190
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/11573#discussion_r55336099
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AlterTableCommandParser.scala
---
@@ -0,0 +1,410 @@
+/*
+ * Licensed to
Github user dilipbiswal commented on the pull request:
https://github.com/apache/spark/pull/11563#issuecomment-193689584
@cloud-fan Thanks..I understand now from Reynold's and your feedback. I
wrongly assumed the goal to make the best effort to retain the original sql
layout.
Github user MLnick commented on the pull request:
https://github.com/apache/spark/pull/11553#issuecomment-193689132
I don't have a strong sense for how much impact a default of `0.01` vs say
`0.05` for the error param would make in terms of performance / accuracy
tradeoff (and memory
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/11573#discussion_r55335658
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkQl.scala ---
@@ -64,6 +85,53 @@ private[sql] class SparkQl(conf: ParserConf =
SimplePar
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11443#issuecomment-193688619
**[Test build #52653 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52653/consoleFull)**
for PR 11443 at commit
[`46e3340`](https://gi
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11573#discussion_r55335240
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AlterTableCommandParser.scala
---
@@ -0,0 +1,410 @@
+/*
+ * Licensed to
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/11573#discussion_r55335131
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AlterTableCommandParser.scala
---
@@ -0,0 +1,410 @@
+/*
+ * Licensed to
601 - 700 of 777 matches
Mail list logo