Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21129
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21129
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90126/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21129
**[Test build #90126 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90126/testReport)**
for PR 21129 at commit
[`95b5497`](https://github.com/apache/spark/commit/95
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21193#discussion_r185794875
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/javaCode.scala
---
@@ -112,6 +112,102 @@ object JavaCode {
de
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21068
**[Test build #90125 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90125/testReport)**
for PR 21068 at commit
[`2a8ab8d`](https://github.com/apache/spark/commit/2a
Github user gaborgsomogyi commented on the issue:
https://github.com/apache/spark/pull/21214
> Just curious how this happened?
I've taken a look at 2 logs you've shown and seen the same pattern. Namely
the test which killed threads passed and right after the upcoming test stuck in
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21193#discussion_r185794699
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/javaCode.scala
---
@@ -120,13 +216,15 @@ object JavaCode {
trai
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21225#discussion_r185793645
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/window/WindowExec.scala
---
@@ -114,7 +114,8 @@ case class WindowExec(
* @para
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21054#discussion_r185791203
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/regression.scala
---
@@ -0,0 +1,190 @@
+/*
+ * Licensed to
Github user juliuszsompolski commented on a diff in the pull request:
https://github.com/apache/spark/pull/21228#discussion_r185792840
--- Diff: R/pkg/R/functions.R ---
@@ -3184,6 +3191,7 @@ setMethod("create_map",
#' collect(select(df2, collect_list(df2$gear)))
#' collect
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21228#discussion_r185792266
--- Diff: R/pkg/R/functions.R ---
@@ -963,6 +964,7 @@ setMethod("kurtosis",
#' last(df$c, TRUE)
#' }
#' @note last since 1.4.0
+#' @no
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/21214#discussion_r185792166
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameRangeSuite.scala ---
@@ -153,23 +153,17 @@ class DataFrameRangeSuite extends QueryTest
Github user juliuszsompolski commented on a diff in the pull request:
https://github.com/apache/spark/pull/21228#discussion_r185791719
--- Diff: R/pkg/R/functions.R ---
@@ -963,6 +964,7 @@ setMethod("kurtosis",
#' last(df$c, TRUE)
#' }
#' @note last since 1.4.0
+#
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185791498
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTrackerSuite.scala
---
@@ -0,0 +1,144 @@
+/*
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21214
> but the thread killing left the shared SparkContext sometimes in a state
where further jobs can't be submitted.
Just curious how this happened?
---
-
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21224
@cloud-fan, the change seems fine but would there be any clever trick to
test this? Seems we could very likely do the similar thing by mistake.
---
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21219#discussion_r185788654
--- Diff:
core/src/main/scala/org/apache/spark/storage/ShuffleBlockFetcherIterator.scala
---
@@ -407,6 +407,25 @@ final class ShuffleBlockFetcherIterat
Github user ala commented on a diff in the pull request:
https://github.com/apache/spark/pull/21227#discussion_r185787003
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/WritableColumnVector.java
---
@@ -81,7 +81,9 @@ public void close() {
}
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21228
**[Test build #90124 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90124/testReport)**
for PR 21228 at commit
[`1761ed9`](https://github.com/apache/spark/commit/17
Github user lokm01 commented on the issue:
https://github.com/apache/spark/pull/21215
@maropu Thanks! Didn't know about creating a literal this way.
Don't you feel that the suggested change is way more elegant?
---
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21228
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21228
add to whitelist
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21228#discussion_r185783283
--- Diff: python/pyspark/sql/functions.py ---
@@ -151,13 +151,15 @@ def _():
_collect_list_doc = """
Aggregate function: returns a li
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21228#discussion_r185783224
--- Diff: R/pkg/R/functions.R ---
@@ -818,6 +818,7 @@ setMethod("factorial",
#' first(df$c, TRUE)
#' }
#' @note first(characterOrColumn) s
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21228#discussion_r185783395
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/misc.scala
---
@@ -115,15 +115,15 @@ case class CurrentDatabase() exten
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21228#discussion_r185784205
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/randomExpressions.scala
---
@@ -94,9 +94,10 @@ object Rand {
}
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21228#discussion_r185784148
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -561,6 +571,7 @@ object functions {
* The function by default retur
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/21054#discussion_r185783983
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameAggregateSuite.scala ---
@@ -36,6 +36,8 @@ case class Fact(date: Int, hour: Int, minute: Int,
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/21054#discussion_r185783832
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameAggregateSuite.scala ---
@@ -686,4 +687,72 @@ class DataFrameAggregateSuite extends QueryTest
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21028#discussion_r185783290
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -19,14 +19,41 @@ package org.apache.spar
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21215
Like this?
```
scala> val structTy = StructType.fromDDL("a ARRAY>")
structTy: org.apache.spark.sql.types.StructType =
StructType(StructField(a,ArrayType(StructType(StructField(b,IntegerTyp
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21228
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21028#discussion_r185781375
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -378,6 +405,127 @@ case class ArrayCont
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21228
**[Test build #90123 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90123/testReport)**
for PR 21228 at commit
[`1761ed9`](https://github.com/apache/spark/commit/17
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/21228
@juliuszsompolski please, look at it.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
GitHub user MaxGekk opened a pull request:
https://github.com/apache/spark/pull/21228
[SPARK-24171] Adding a note for non-deterministic functions
## What changes were proposed in this pull request?
I propose to add a clear statement for functions like `collect_list()`
about
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21028#discussion_r185780243
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -19,14 +19,41 @@ package org.apache.spa
Github user lokm01 commented on the issue:
https://github.com/apache/spark/pull/21215
@maropu That would work if you had scala case classes for all the types. In
our case, we're working on a generic framework, where we only have Spark
schemas (and I'd rather not generate case classes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20947
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2865/
Tes
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21223
thanks, merging to master/2.3!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-ma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20947
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21214
**[Test build #90122 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90122/testReport)**
for PR 21214 at commit
[`9781cbe`](https://github.com/apache/spark/commit/97
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21070
Simple scan benchmarks:
code:
https://github.com/apache/spark/compare/master...maropu:DataSourceReadBenchmark
master: https://gist.github.com/maropu/a767d21ed1dd047ec2bdca92915dc5c5
this p
Github user gaborgsomogyi commented on the issue:
https://github.com/apache/spark/pull/21214
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revi
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21223
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21054
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21054
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2864/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21054
**[Test build #90121 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90121/testReport)**
for PR 21054 at commit
[`e584adf`](https://github.com/apache/spark/commit/e5
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21182#discussion_r185773095
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JSONOptions.scala
---
@@ -136,4 +138,6 @@ private[sql] class JSONOptions(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21219
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90112/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21219
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21219
**[Test build #90112 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90112/testReport)**
for PR 21219 at commit
[`41d06e1`](https://github.com/apache/spark/commit/4
Github user edwinalu closed the pull request at:
https://github.com/apache/spark/pull/20940
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/21227#discussion_r185768453
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/WritableColumnVector.java
---
@@ -81,7 +81,9 @@ public void close() {
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21169
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20973
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20973
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90116/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20973
**[Test build #90116 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90116/testReport)**
for PR 20973 at commit
[`76d4119`](https://github.com/apache/spark/commit/7
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21169
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21163
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90115/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21163
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21193
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21193
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2863/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21163
**[Test build #90115 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90115/testReport)**
for PR 21163 at commit
[`e303770`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21169
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21169
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90108/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21169
**[Test build #90108 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90108/testReport)**
for PR 21169 at commit
[`b6d91db`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21227
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2862/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21227
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21222
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21222
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90111/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21193
**[Test build #90120 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90120/testReport)**
for PR 21193 at commit
[`00bef6b`](https://github.com/apache/spark/commit/00
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21227
**[Test build #90119 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90119/testReport)**
for PR 21227 at commit
[`638aea7`](https://github.com/apache/spark/commit/63
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21169
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90105/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21222
**[Test build #90111 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90111/testReport)**
for PR 21222 at commit
[`c1ad1c5`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21169
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user ala commented on the issue:
https://github.com/apache/spark/pull/21227
@gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21169
**[Test build #90105 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90105/testReport)**
for PR 21169 at commit
[`b6d91db`](https://github.com/apache/spark/commit/b
GitHub user ala opened a pull request:
https://github.com/apache/spark/pull/21227
[SPARK-24133][SQL] Check for integer overflows when resizing
WritableColumnVectors
`ColumnVector`s store string data in one big byte array. Since the array
size is capped at just under Integer.MAX_VAL
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21201
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90109/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21021
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21201
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21021
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2861/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21201
**[Test build #90109 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90109/testReport)**
for PR 21201 at commit
[`42d0241`](https://github.com/apache/spark/commit/4
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21021
**[Test build #90118 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90118/testReport)**
for PR 21021 at commit
[`9f63a76`](https://github.com/apache/spark/commit/9f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21223
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21223
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90107/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21224
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90110/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21224
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21223
**[Test build #90107 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90107/testReport)**
for PR 21223 at commit
[`d900b4c`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21224
**[Test build #90110 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90110/testReport)**
for PR 21224 at commit
[`d7dc8a8`](https://github.com/apache/spark/commit/d
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/21021
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21193#discussion_r185761453
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/javaCode.scala
---
@@ -120,13 +216,15 @@ object JavaCode {
tr
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21193#discussion_r185761604
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/javaCode.scala
---
@@ -112,6 +112,102 @@ object JavaCode {
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21193#discussion_r185763081
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/javaCode.scala
---
@@ -112,6 +112,102 @@ object JavaCode {
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21193#discussion_r185762835
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/javaCode.scala
---
@@ -112,6 +112,102 @@ object JavaCode {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21193
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21193
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90104/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21193
**[Test build #90104 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90104/testReport)**
for PR 21193 at commit
[`5fe425c`](https://github.com/apache/spark/commit/5
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/21182#discussion_r185757247
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JSONOptions.scala
---
@@ -136,4 +138,6 @@ private[sql] class JSONOptions(
501 - 600 of 715 matches
Mail list logo