Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/12647#issuecomment-213858553
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
enable
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12645#issuecomment-213857179
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/12645#issuecomment-213857174
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/11956#issuecomment-213857098
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12645#issuecomment-213857074
**[Test build #56818 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56818/consoleFull)**
for PR 12645 at commit
[`9053334`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11956#issuecomment-213857099
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/11956#issuecomment-213857070
**[Test build #56817 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56817/consoleFull)**
for PR 11956 at commit
[`2abd436`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12647#issuecomment-213857058
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/12647#issuecomment-213857059
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/12647#issuecomment-213857042
**[Test build #56821 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56821/consoleFull)**
for PR 12647 at commit
[`aeceabd`](https://g
Github user felixcheung commented on the pull request:
https://github.com/apache/spark/pull/12648#issuecomment-213856851
# spark-shell
```
# ./bin/spark-shell
Using Spark's default log4j profile:
org/apache/spark/log4j-defaults.properties
Setting default log l
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12648#issuecomment-213856690
**[Test build #56822 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56822/consoleFull)**
for PR 12648 at commit
[`005892a`](https://gi
GitHub user felixcheung opened a pull request:
https://github.com/apache/spark/pull/12648
[SPARK-14881][PYTHON][SPARKR] pyspark and sparkR shell default log level
should match spark-shell/Scala
## What changes were proposed in this pull request?
Change default logging to WA
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12645#issuecomment-213856567
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/12645#issuecomment-213856566
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/12645#issuecomment-213856283
**[Test build #56816 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56816/consoleFull)**
for PR 12645 at commit
[`2e18c61`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12647#issuecomment-213855988
**[Test build #56821 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56821/consoleFull)**
for PR 12647 at commit
[`aeceabd`](https://gi
Github user felixcheung commented on the pull request:
https://github.com/apache/spark/pull/12647#issuecomment-213855153
my bad, I should have checked more carefully, thanks for finding this
@shivaram.
@rxin
---
If your project is set up for it, you can reply to this email and h
GitHub user felixcheung opened a pull request:
https://github.com/apache/spark/pull/12647
[SPARK-12148][SPARKR] fix doc after renaming DataFrame to SparkDataFrame
## What changes were proposed in this pull request?
Fixed inadvertent roxygen2 doc changes, added class name cha
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11956#issuecomment-213855047
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/11956#issuecomment-213855045
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/11956#issuecomment-213854956
**[Test build #56819 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56819/consoleFull)**
for PR 11956 at commit
[`f09ed72`](https://g
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/12259#discussion_r60836845
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala ---
@@ -117,6 +118,12 @@ class SQLContext private[sql](
*
* @since 1.6.
Github user felixcheung commented on the pull request:
https://github.com/apache/spark/pull/11569#issuecomment-213854068
please rebase to pick up `DataFrame` -> `SparkDataFrame` class name change.
---
If your project is set up for it, you can reply to this email and have your
reply ap
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12599#issuecomment-213853964
**[Test build #56820 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56820/consoleFull)**
for PR 12599 at commit
[`6b6c12d`](https://gi
Github user kevinyu98 commented on the pull request:
https://github.com/apache/spark/pull/12646#issuecomment-213853674
@hvanhovell @yhuai @chenghao-intel @gatorsmile @dilipbiswal @viirya
@xwu0226 can you help take a look at this PR? Thanks.
---
If your project is set up for it, you
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12639
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/12642
---
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 viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/12599#discussion_r60836715
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/object.scala
---
@@ -83,6 +83,28 @@ case class SerializeFromObject(
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/12639#issuecomment-213853550
Merging this into master, thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does no
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12643#issuecomment-213853109
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 davies commented on the pull request:
https://github.com/apache/spark/pull/12642#issuecomment-213853108
Merging this into master, thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does no
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12643#issuecomment-213853110
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/12643#issuecomment-213853096
**[Test build #56814 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56814/consoleFull)**
for PR 12643 at commit
[`172a31a`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12646#issuecomment-213853045
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 kevinyu98 opened a pull request:
https://github.com/apache/spark/pull/12646
[SPARK-14878][SQL] Trim characters string function support
What changes were proposed in this pull request?
This PR enhances the TRIM function support in Spark SQL by
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11956#issuecomment-213851558
**[Test build #56819 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56819/consoleFull)**
for PR 11956 at commit
[`f09ed72`](https://gi
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/12626#discussion_r60836257
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala
---
@@ -71,7 +71,9 @@ object CurrentOrigin {
}
}
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12645#issuecomment-213850298
**[Test build #56818 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56818/consoleFull)**
for PR 12645 at commit
[`9053334`](https://gi
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12644
---
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 yhuai commented on the pull request:
https://github.com/apache/spark/pull/12644#issuecomment-213849631
Thanks! Merging 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 t
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12644#issuecomment-213849530
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/12644#issuecomment-213849531
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/11956#issuecomment-213849521
**[Test build #56817 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56817/consoleFull)**
for PR 11956 at commit
[`2abd436`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12645#issuecomment-213849516
**[Test build #56816 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56816/consoleFull)**
for PR 12645 at commit
[`2e18c61`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12644#issuecomment-213849482
**[Test build #56813 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56813/consoleFull)**
for PR 12644 at commit
[`cd9784e`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12645#issuecomment-213848524
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/12645#issuecomment-213848523
**[Test build #56815 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56815/consoleFull)**
for PR 12645 at commit
[`95b61bc`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12645#issuecomment-213848525
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/12645#issuecomment-213847597
**[Test build #56815 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56815/consoleFull)**
for PR 12645 at commit
[`95b61bc`](https://gi
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/12645#discussion_r60835970
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala
---
@@ -0,0 +1,436 @@
+/*
+ * Licensed to the Ap
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/12645#discussion_r60835975
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala
---
@@ -0,0 +1,436 @@
+/*
+ * Licensed to the Ap
GitHub user yhuai opened a pull request:
https://github.com/apache/spark/pull/12645
[SPARK-14879] [SQL] Move CreateMetastoreDataSource and
CreateMetastoreDataSourceAsSelect to sql/core
## What changes were proposed in this pull request?
CreateMetastoreDataSource and CreateM
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12643#issuecomment-213845119
**[Test build #56814 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56814/consoleFull)**
for PR 12643 at commit
[`172a31a`](https://gi
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/12644#issuecomment-213842068
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 an
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12643#issuecomment-213841991
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/12643#issuecomment-213841993
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/12643#issuecomment-213841958
**[Test build #56812 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56812/consoleFull)**
for PR 12643 at commit
[`af6ff04`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12644#issuecomment-213835275
**[Test build #56813 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56813/consoleFull)**
for PR 12644 at commit
[`cd9784e`](https://gi
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12644#issuecomment-213834903
cc @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
enable
GitHub user rxin opened a pull request:
https://github.com/apache/spark/pull/12644
[SPARK-14877][SQL] Remove HiveMetastoreTypes class
## What changes were proposed in this pull request?
It is unnecessary as DataType.catalogString largely replaces the need for
this class.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12642#issuecomment-213833844
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 rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12641#discussion_r60835072
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/YarnClusterManager.scala ---
@@ -0,0 +1,80 @@
+/*
+ * Licensed to the Apache Software Foundatio
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12642#issuecomment-213833843
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 rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12641#discussion_r60835060
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/YarnClusterManager.scala ---
@@ -0,0 +1,80 @@
+/*
+ * Licensed to the Apache Software Foundatio
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12641#discussion_r60835052
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/YarnClusterManager.scala ---
@@ -0,0 +1,80 @@
+/*
+ * Licensed to the Apache Software Foundatio
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12642#issuecomment-213833795
**[Test build #56811 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56811/consoleFull)**
for PR 12642 at commit
[`40bed84`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12631#issuecomment-213832629
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/12631#issuecomment-213832634
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/12631#issuecomment-213832531
**[Test build #56810 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56810/consoleFull)**
for PR 12631 at commit
[`d593607`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12643#issuecomment-213831538
**[Test build #56812 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56812/consoleFull)**
for PR 12643 at commit
[`af6ff04`](https://gi
GitHub user rxin opened a pull request:
https://github.com/apache/spark/pull/12643
[SPARK-14876][SQL] SparkSession should be case insensitive by default
## What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
## How was
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12641#discussion_r60834795
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/YarnClusterManager.scala ---
@@ -0,0 +1,80 @@
+/*
+ * Licensed to the Apache Software Foundatio
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12641#issuecomment-213831188
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 feature
enab
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12641#issuecomment-213829463
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/12641#issuecomment-213829467
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/12641#issuecomment-213828846
**[Test build #56807 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56807/consoleFull)**
for PR 12641 at commit
[`643f8d2`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11886#issuecomment-213825500
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/11886#issuecomment-213825501
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/11886#issuecomment-213825430
**[Test build #56806 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56806/consoleFull)**
for PR 11886 at commit
[`8f8133c`](https://g
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12633
---
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 yhuai commented on the pull request:
https://github.com/apache/spark/pull/12633#issuecomment-213823889
LGTM. Merging to master. Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/12631#discussion_r60834332
--- Diff: build/mvn ---
@@ -69,14 +69,23 @@ install_app() {
# Install maven under the build/ folder
install_mvn() {
- local
APACHE
Github user markhamstra commented on a diff in the pull request:
https://github.com/apache/spark/pull/12631#discussion_r60834085
--- Diff: build/mvn ---
@@ -69,14 +69,23 @@ install_app() {
# Install maven under the build/ folder
install_mvn() {
- local
APACHE_M
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/12631#discussion_r60834026
--- Diff: build/mvn ---
@@ -69,14 +69,23 @@ install_app() {
# Install maven under the build/ folder
install_mvn() {
- local
APACHE
Github user jaceklaskowski commented on the pull request:
https://github.com/apache/spark/pull/12569#issuecomment-213819194
Could this PR have a bit of @srowen's attention? :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12634
---
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 dongjoon-hyun commented on the pull request:
https://github.com/apache/spark/pull/12642#issuecomment-213818944
Hi, @davies .
Could you review this minor PR? It's a fix for code made by you at
[6bba750](https://github.com/apache/spark/commit/6bba7509a932aa4d39266df2d15b1
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12634#issuecomment-213818904
Merging in 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 feat
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12642#issuecomment-213817806
**[Test build #56811 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56811/consoleFull)**
for PR 12642 at commit
[`40bed84`](https://gi
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12636
---
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 dongjoon-hyun opened a pull request:
https://github.com/apache/spark/pull/12642
[MINOR][SQL] Fix error message string in nullSafeEvel of TernaryExpression
## What changes were proposed in this pull request?
TernaryExpressions should thows proper error message for
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12635
---
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 markhamstra commented on a diff in the pull request:
https://github.com/apache/spark/pull/12631#discussion_r60833824
--- Diff: build/mvn ---
@@ -69,14 +69,23 @@ install_app() {
# Install maven under the build/ folder
install_mvn() {
- local
APACHE_M
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12634#issuecomment-213817224
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/12634#issuecomment-213817223
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 yhuai commented on the pull request:
https://github.com/apache/spark/pull/12636#issuecomment-213817228
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 an
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/12635#issuecomment-213817153
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 an
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12634#issuecomment-213817154
**[Test build #56805 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56805/consoleFull)**
for PR 12634 at commit
[`cddc66b`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12631#issuecomment-213807297
**[Test build #56810 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56810/consoleFull)**
for PR 12631 at commit
[`d593607`](https://gi
101 - 200 of 380 matches
Mail list logo