Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21814
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21815
**[Test build #93281 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93281/testReport)**
for PR 21815 at commit
[`b488836`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21815
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/21815
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93281/
Test PASSed.
---
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21814
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21803
Ah, I misunderstood then. Thing is, `fromDDL` was added because it's needed
other APIs rather then it's own purpose. I wonder how commonly it will be used
to be honest. I haven't seen such reque
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21798
**[Test build #93283 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93283/testReport)**
for PR 21798 at commit
[`3bd3475`](https://github.com/apache/spark/commit/3b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21798
**[Test build #93283 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93283/testReport)**
for PR 21798 at commit
[`3bd3475`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21798
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93283/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21798
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21729
+1 I'm going to merge, thanks @hthuynh2
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comm
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21729
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21653
#21729 has been merged @hthuynh2 can you update this one?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
F
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/21803
> Thing is, fromDDL was added because it's needed other APIs rather then
it's own purpose.
The `toDDL` function can be reused from `SHOW CREATE TABLE` too:
https://github.com/apache/spark/
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21546#discussion_r203760078
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -3349,20 +3385,20 @@ class Dataset[T] private[sql](
}
}
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/21798
I added new option to `AvroOptions` and documented it. Please, look at the
PR one more time.
---
-
To unsubscribe, e-mail: revie
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21803
To me yup, but if you are in doubt, I am perfectly okay with waiting some
more days and see if other opinions arrive.
---
-
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21103
**[Test build #93280 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93280/testReport)**
for PR 21103 at commit
[`26e3257`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21103
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/21103
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93280/
Test PASSed.
---
Github user mn-mikke commented on a diff in the pull request:
https://github.com/apache/spark/pull/21103#discussion_r203741325
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3805,3 +3799,332 @@ object ArrayUnion {
Github user mn-mikke commented on a diff in the pull request:
https://github.com/apache/spark/pull/21103#discussion_r203746577
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3805,3 +3799,332 @@ object ArrayUnion {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21758
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-unified/1130/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21758
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/21758
**[Test build #93284 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93284/testReport)**
for PR 21758 at commit
[`ed89af8`](https://github.com/apache/spark/commit/ed
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21440#discussion_r203773818
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -659,6 +659,11 @@ private[spark] class BlockManager(
* Get block fro
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21758
**[Test build #93284 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93284/testReport)**
for PR 21758 at commit
[`ed89af8`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21758
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93284/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21758
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user Achuth17 commented on the issue:
https://github.com/apache/spark/pull/21608
retest this please.
---
-
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/21804
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-unified/1131/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21804
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/21804
**[Test build #93285 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93285/testReport)**
for PR 21804 at commit
[`20ee744`](https://github.com/apache/spark/commit/20
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21546#discussion_r203790304
--- Diff: python/pyspark/serializers.py ---
@@ -184,27 +184,67 @@ def loads(self, obj):
raise NotImplementedError
-class Arr
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20761
**[Test build #93286 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93286/testReport)**
for PR 20761 at commit
[`55dfc98`](https://github.com/apache/spark/commit/55
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21782
I don't have a good idea, cc @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comm
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r203790741
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeValidator.scala
---
@@ -0,0 +1,185 @@
+/*
+ * Licensed to t
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20761
**[Test build #93286 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93286/testReport)**
for PR 20761 at commit
[`55dfc98`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20761
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93286/
Test FAILed.
---
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r203791125
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeValidator.scala
---
@@ -0,0 +1,185 @@
+/*
+ * Licensed to t
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20761
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user szyszy commented on the issue:
https://github.com/apache/spark/pull/20761
About the ResourceTypeValidator: The reason why this class seems complex is
that it is capable of printing "smart" error messages like they are described
in the scaladoc.
When you define memory w
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r203791602
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala
---
@@ -249,6 +259,10 @@ private[spark] class Client(
val
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r203791726
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeValidator.scala
---
@@ -0,0 +1,185 @@
+/*
+ * Licensed to t
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21758
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-unified/1132/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21758
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/21758
**[Test build #93287 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93287/testReport)**
for PR 21758 at commit
[`51bbb0f`](https://github.com/apache/spark/commit/51
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21546
@BryanCutler, this takes longer then I thought. Will complete my review
till this week. For clarification, still no objection about merging it in
orthogonally with my review.
---
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r203793303
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeHelper.scala
---
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r203793589
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeHelper.scala
---
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r203793778
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeValidator.scala
---
@@ -0,0 +1,185 @@
+/*
+ * Licensed to t
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r203793702
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeHelper.scala
---
@@ -0,0 +1,150 @@
+/*
+ * Licensed to the
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r203793839
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeValidator.scala
---
@@ -0,0 +1,185 @@
+/*
+ * Licensed to t
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r203793921
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala
---
@@ -306,9 +318,14 @@ private[yarn] class YarnAllocator(
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r203793998
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala
---
@@ -306,9 +318,14 @@ private[yarn] class YarnAllocator(
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r203794300
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala
---
@@ -17,7 +17,7 @@
package org.apache.spark.deploy.y
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r203794227
--- Diff: docs/running-on-yarn.md ---
@@ -121,6 +121,40 @@ To use a custom metrics.properties for the application
master and executors, upd
Use lowe
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r203794112
--- Diff: docs/running-on-yarn.md ---
@@ -121,6 +121,40 @@ To use a custom metrics.properties for the application
master and executors, upd
Use lowe
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r203794078
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala
---
@@ -306,9 +318,14 @@ private[yarn] class YarnAllocator(
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r203794562
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeHelper.scala
---
@@ -0,0 +1,150 @@
+/*
+ * Licensed to the
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r203794443
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeHelper.scala
---
@@ -0,0 +1,150 @@
+/*
+ * Licensed to the
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r203794460
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeHelper.scala
---
@@ -0,0 +1,150 @@
+/*
+ * Licensed to the
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r203794842
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeHelper.scala
---
@@ -0,0 +1,150 @@
+/*
+ * Licensed to the
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r203794896
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeValidator.scala
---
@@ -0,0 +1,185 @@
+/*
+ * Licensed to t
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r203794871
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeValidator.scala
---
@@ -0,0 +1,185 @@
+/*
+ * Licensed to t
Github user szyszy commented on the issue:
https://github.com/apache/spark/pull/20761
Hi @vanzin , @galv
Please check my latest fixes based on your previous comments.
@squito : Could you please join this review to speed things up?
Thanks in advance!
---
-
Github user arunmahadevan commented on the issue:
https://github.com/apache/spark/pull/21721
@jose-torres, addressed initial comments.
@tdas, can you also take a look when possible ?
---
-
To unsubscribe, e-mail:
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/21774#discussion_r203796377
--- Diff:
external/avro/src/main/scala/org/apache/spark/sql/avro/CatalystDataToAvro.scala
---
@@ -0,0 +1,67 @@
+/*
+ * Licensed to the Apache
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21774
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/21774
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-unified/1133/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21774
**[Test build #93288 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93288/testReport)**
for PR 21774 at commit
[`833dccc`](https://github.com/apache/spark/commit/83
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21758
**[Test build #93287 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93287/testReport)**
for PR 21758 at commit
[`51bbb0f`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21758
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/21758
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93287/
Test FAILed.
---
GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/21817
[SPARK-24861][SS] create corrected temp directories in RateSourceSuite
## What changes were proposed in this pull request?
`RateSourceSuite` may leave garbage files under `sql/core/dummy`
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21817
cc @jose-torres @tdas
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21817
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/21817
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-unified/1134/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21817
**[Test build #93289 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93289/testReport)**
for PR 21817 at commit
[`10c5ecf`](https://github.com/apache/spark/commit/10
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21774#discussion_r203805491
--- Diff:
external/avro/src/main/scala/org/apache/spark/sql/avro/CatalystDataToAvro.scala
---
@@ -0,0 +1,67 @@
+/*
+ * Licensed to the Apache Sof
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21774#discussion_r203805849
--- Diff:
external/avro/src/test/scala/org/apache/spark/sql/avro/AvroFunctionsSuite.scala
---
@@ -0,0 +1,87 @@
+/*
+ * Licensed to the Apache Sof
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21774
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apa
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21320
@mallman Could we move the changes made in `ParquetReadSupport.scala` to a
separate PR? Then, we can merge this PR very quickly.
---
---
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21813#discussion_r203813530
--- Diff:
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
@@ -400,12 +400,12 @@ hintStatement
fromClause
: FRO
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21813#discussion_r203813381
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -440,19 +440,36 @@ class Analyzer(
groupByEx
GitHub user koertkuipers opened a pull request:
https://github.com/apache/spark/pull/21818
[SPARK-24860][SQL] Support setting of partitionOverWriteMode in output
options for writing DataFrame
## What changes were proposed in this pull request?
Besides spark setting spark.sq
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21818
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-unified/1135/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21818
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user koertkuipers commented on the issue:
https://github.com/apache/spark/pull/18714
@cloud-fan i created
[SPARK-24860](https://issues.apache.org/jira/browse/SPARK-24860) for this
---
-
To unsubscribe, e-mail
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21818
**[Test build #93290 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93290/testReport)**
for PR 21818 at commit
[`7dd2eab`](https://github.com/apache/spark/commit/7d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21813
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-unified/1136/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21813
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/21813
**[Test build #93291 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93291/testReport)**
for PR 21813 at commit
[`ac8f04f`](https://github.com/apache/spark/commit/ac
Github user pgandhi999 commented on a diff in the pull request:
https://github.com/apache/spark/pull/15703#discussion_r203828759
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/hiveUDFs.scala
---
@@ -365,4 +394,74 @@ private[hive] case class HiveUDAFFunction(
v
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21774
**[Test build #93292 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93292/testReport)**
for PR 21774 at commit
[`a6dd08f`](https://github.com/apache/spark/commit/a6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21774
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-unified/1137/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21774
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user ifilonenko commented on the issue:
https://github.com/apache/spark/pull/21584
All tests pass locally. Waiting on PRB to be configured.
```
KubernetesSuite:
- Run SparkPi with no resources
- Run SparkPi with a very long application name.
- Use SparkLaunch
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21635#discussion_r203840572
--- Diff: docs/monitoring.md ---
@@ -435,6 +435,7 @@ set of sinks to which metrics are reported. The
following instances are currentl
* `executor`:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21584
**[Test build #93294 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93294/testReport)**
for PR 21584 at commit
[`d809f67`](https://github.com/apache/spark/commit/d8
201 - 300 of 529 matches
Mail list logo