Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13964
**[Test build #3141 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3141/consoleFull)**
for PR 13964 at commit
[`8d2db6e`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13860
**[Test build #61453 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61453/consoleFull)**
for PR 13860 at commit
[`e467d74`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13965
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61447/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13965
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13965
**[Test build #61447 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61447/consoleFull)**
for PR 13965 at commit
[`9cfd673`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13966
**[Test build #3139 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3139/consoleFull)**
for PR 13966 at commit
[`7cea3b1`](https://github.com/apache/spark/commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13860
: ) Just found one in `AstBuilder.scala`. Let me know if anything I still
missed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13933
@zsxwing If we just provide one `path` in the function input, it will not
put `path` into the `options`. The API `parquet(path: String)`still calls
`load(paths : _*)`, instead of `load(path)`. Th
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13829#discussion_r68891055
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/codegen/BufferHolder.java
---
@@ -55,6 +61,11 @@ public BufferHolder(Unsafe
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13829
**[Test build #61452 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61452/consoleFull)**
for PR 13829 at commit
[`336986d`](https://github.com/apache/spark/commit/3
Github user clockfly commented on a diff in the pull request:
https://github.com/apache/spark/pull/13829#discussion_r68890765
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/codegen/BufferHolder.java
---
@@ -45,7 +45,13 @@ public BufferHolder(UnsafeR
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13966#discussion_r68890695
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -162,6 +162,42 @@ case class ConcatWs(child
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/13964
LGTM, pending tests, thanks for working on it!
---
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 issue:
https://github.com/apache/spark/pull/13964
**[Test build #3138 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3138/consoleFull)**
for PR 13964 at commit
[`34cda07`](https://github.com/apache/spark/commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/13860
Seems there are more places that we can use the new `addComment`, can you
run `grep "\"comment\"" -rn sql` and double check them? thanks
---
If your project is set up for it, you can reply to thi
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/13948
Merged to 2.0 as well.
---
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 s
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13860#discussion_r68889544
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/test/DataFrameReaderWriterSuite.scala
---
@@ -363,6 +363,31 @@ class DataFrameReaderWriterSuite ext
Github user petermaxlee commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68889173
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/xml/XPathExpressionSuite.scala
---
@@ -0,0 +1,68 @@
+/*
+ * Lice
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13930
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13930
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61446/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13930
**[Test build #61446 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61446/consoleFull)**
for PR 13930 at commit
[`b8df028`](https://github.com/apache/spark/commit/
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r6729
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/xml/XPathExpressionSuite.scala
---
@@ -0,0 +1,68 @@
+/*
+ * Licens
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13860
**[Test build #61451 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61451/consoleFull)**
for PR 13860 at commit
[`b6ded4d`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13886
**[Test build #61450 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61450/consoleFull)**
for PR 13886 at commit
[`e4cc35d`](https://github.com/apache/spark/commit/e
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/13860#discussion_r6152
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/types/DataTypeSuite.scala ---
@@ -52,6 +52,23 @@ class DataTypeSuite extends SparkFunSuite {
Github user petermaxlee commented on a diff in the pull request:
https://github.com/apache/spark/pull/13966#discussion_r6095
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -162,6 +162,42 @@ case class ConcatWs(chi
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13860#discussion_r6077
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/types/DataTypeSuite.scala ---
@@ -52,6 +52,23 @@ class DataTypeSuite extends SparkFunSuite {
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r6030
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/FunctionRegistry.scala
---
@@ -301,6 +302,7 @@ object FunctionRegistry {
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/13860#discussion_r6029
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/test/DataFrameReaderWriterSuite.scala
---
@@ -363,6 +363,31 @@ class DataFrameReaderWriterSuite ex
Github user petermaxlee commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r6009
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/xml/XPathExpressionSuite.scala
---
@@ -0,0 +1,68 @@
+/*
+ * Lice
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13860#discussion_r68887957
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/types/DataTypeSuite.scala ---
@@ -52,6 +52,23 @@ class DataTypeSuite extends SparkFunSuite {
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13886
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 feature
enabled and wishes so
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/13860#discussion_r68887844
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/types/DataTypeSuite.scala ---
@@ -52,6 +52,23 @@ class DataTypeSuite extends SparkFunSuite {
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/13906
For 3, I respect your opinion. I just make another commit for 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
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68887731
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/FunctionRegistry.scala
---
@@ -301,6 +302,7 @@ object FunctionRegistry {
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68887674
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/xml/XPathExpressionSuite.scala
---
@@ -0,0 +1,68 @@
+/*
+ * Licens
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13919
**[Test build #61448 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61448/consoleFull)**
for PR 13919 at commit
[`b999b8a`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13906
**[Test build #61449 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61449/consoleFull)**
for PR 13906 at commit
[`4d937dc`](https://github.com/apache/spark/commit/4
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13966#discussion_r68887526
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -162,6 +162,42 @@ case class ConcatWs(child
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/13893
It's a good point, looks like we can also improve the `PushDownPredicate`
rule according to this.
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13912
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13912
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61443/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13912
**[Test build #61443 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61443/consoleFull)**
for PR 13912 at commit
[`0e1901e`](https://github.com/apache/spark/commit/
Github user petermaxlee commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68887194
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/FunctionRegistry.scala
---
@@ -301,6 +302,7 @@ object FunctionRegistry {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13912
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13912
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61444/
Test FAILed.
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13860#discussion_r68887161
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/test/DataFrameReaderWriterSuite.scala
---
@@ -363,6 +363,31 @@ class DataFrameReaderWriterSuite ext
Github user petermaxlee commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68887178
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/xml/XPathBoolean.scala
---
@@ -0,0 +1,56 @@
+/*
+ * Licensed to
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13912
**[Test build #61444 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61444/consoleFull)**
for PR 13912 at commit
[`d03e7a0`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13966
**[Test build #3139 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3139/consoleFull)**
for PR 13966 at commit
[`7cea3b1`](https://github.com/apache/spark/commit/
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13860#discussion_r68886986
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/types/DataTypeSuite.scala ---
@@ -52,6 +52,23 @@ class DataTypeSuite extends SparkFunSuite {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13964
**[Test build #3140 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3140/consoleFull)**
for PR 13964 at commit
[`bdd49aa`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13966
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feat
Github user petermaxlee commented on the issue:
https://github.com/apache/spark/pull/13966
cc @dongjoon-hyun @cloud-fan @rxin
---
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 petermaxlee opened a pull request:
https://github.com/apache/spark/pull/13966
[SPARK-16276][SQL] Implement elt SQL function
## What changes were proposed in this pull request?
This patch implements the elt function, as it is implemented in Hive.
## How was th
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/13893
Predicates should not be reordered if a condition contains
non-deterministic parts, for example, 'rand() < 0.1 AND a=1' should not be
reordered to 'a=1 AND rand() < 0.1' as the number of calls
Github user gurvindersingh commented on a diff in the pull request:
https://github.com/apache/spark/pull/13950#discussion_r68886506
--- Diff: core/src/main/scala/org/apache/spark/deploy/master/Master.scala
---
@@ -127,7 +128,14 @@ private[deploy] class Master(
logInfo(s"Ru
Github user gurvindersingh commented on a diff in the pull request:
https://github.com/apache/spark/pull/13950#discussion_r68886448
--- Diff: core/src/main/scala/org/apache/spark/deploy/master/Master.scala
---
@@ -127,7 +128,14 @@ private[deploy] class Master(
logInfo(s"Ru
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68886429
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/NonFoldableLiteral.scala
---
@@ -26,7 +26,7 @@ import org.apache.spark.sql
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/13948
Feel free to do it - but please take another careful look before you cherry
pick.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If yo
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/13933
@gatorsmile parquet, json or other file formats support both `path` and
`paths` options. So that's not a problem.
---
If your project is set up for it, you can reply to this email and have your
rep
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68886317
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/xml/XPathBoolean.scala
---
@@ -0,0 +1,56 @@
+/*
+ * Licensed to th
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68886279
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/xml/XPathBoolean.scala
---
@@ -0,0 +1,56 @@
+/*
+ * Licensed to th
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/13906
By the way, for complexity, it's 23 line optimizer without blank/comments.
In fact, it's less than `NullPropagation`.
---
If your project is set up for it, you can reply to this email and hav
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/13948
@rxin why don't we merge this one 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 feature
e
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/13955
---
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 issue:
https://github.com/apache/spark/pull/13906
It sounds promising. Maybe, Spark 2.1?
---
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 zsxwing commented on the issue:
https://github.com/apache/spark/pull/13955
LGTM. Merging to master and 2.0. 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 have this feature
en
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/13906
Sounds interesting. You mean `LocalNode` that computes all local node
operators on `LocalRelation`, right?
---
If your project is set up for it, you can reply to this email and have your
repl
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/13893
no, the predicates order doesn't matter. Our optimizer can reorder the
predicates to run them more efficient.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13704#discussion_r68885567
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Cast.scala
---
@@ -837,8 +837,36 @@ case class Cast(child: Expression, dat
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/13906
I'm not sure if this optimization is useful
1. empty `LocalRelation` is a corner case and seems not worth to optimize.
2. the optimization rule in this PR is kind of complex.
3. if
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/13893
@cloud-fan I pushed a commit to apply predicate pushdown on deterministic
parts placed before any non-deterministic predicates, should it be safe to do
this optimizationï¼
---
If your project
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68884892
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/FunctionRegistry.scala
---
@@ -301,6 +302,7 @@ object FunctionRegistry {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13965
**[Test build #61447 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61447/consoleFull)**
for PR 13965 at commit
[`9cfd673`](https://github.com/apache/spark/commit/9
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/13965
[SPARK-16236] [SQL] [FOLLOWUP] Add Path Option back to Load API in
DataFrameReader
What changes were proposed in this pull request?
When users only specify one and only one path, we use
Github user petermaxlee commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68884564
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/XmlFunctionsSuite.scala ---
@@ -0,0 +1,33 @@
+/*
+ * Licensed to the Apache Software Found
Github user petermaxlee commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68884371
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/XmlFunctionsSuite.scala ---
@@ -0,0 +1,33 @@
+/*
+ * Licensed to the Apache Software Found
Github user petermaxlee commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68884363
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/NonFoldableLiteral.scala
---
@@ -26,7 +26,7 @@ import org.apache.spark.s
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68884303
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/XmlFunctionsSuite.scala ---
@@ -0,0 +1,33 @@
+/*
+ * Licensed to the Apache Software Foundat
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68884274
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/XmlFunctionsSuite.scala ---
@@ -0,0 +1,33 @@
+/*
+ * Licensed to the Apache Software Foundat
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11863
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11863
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61436/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13961
**[Test build #3137 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3137/consoleFull)**
for PR 13961 at commit
[`90bf2f1`](https://github.com/apache/spark/commit
Github user petermaxlee commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68884185
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/xml/XPathExpressionSuite.scala
---
@@ -0,0 +1,68 @@
+/*
+ * Lice
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11863
**[Test build #61436 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61436/consoleFull)**
for PR 11863 at commit
[`b1eec57`](https://github.com/apache/spark/commit/
Github user petermaxlee commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68884156
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/xml/XPathBoolean.scala
---
@@ -0,0 +1,56 @@
+/*
+ * Licensed to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13778
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61441/
Test FAILed.
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68884143
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/NonFoldableLiteral.scala
---
@@ -26,7 +26,7 @@ import org.apache.spark.sql
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13778
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13778
**[Test build #61441 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61441/consoleFull)**
for PR 13778 at commit
[`65a33b0`](https://github.com/apache/spark/commit/
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68884058
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/NonFoldableLiteral.scala
---
@@ -26,7 +26,7 @@ import org.apache.spark
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13963
**[Test build #61445 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61445/consoleFull)**
for PR 13963 at commit
[`a314e56`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13963
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68884017
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/xml/XPathBoolean.scala
---
@@ -0,0 +1,56 @@
+/*
+ * Licensed to th
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13963
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61445/
Test PASSed.
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68883944
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/xml/XPathBoolean.scala
---
@@ -0,0 +1,56 @@
+/*
+ * Licensed to th
Github user petermaxlee commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68883895
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/NonFoldableLiteral.scala
---
@@ -26,7 +26,7 @@ import org.apache.spark.s
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/13778
@cloud-fan @vlad17 Is this change good for your now? 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 h
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/13964#discussion_r68883797
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/NonFoldableLiteral.scala
---
@@ -26,7 +26,7 @@ import org.apache.spark
1 - 100 of 773 matches
Mail list logo