Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22637
**[Test build #96975 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96975/testReport)**
for PR 22637 at commit
[`58de6a3`](https://github.com/apache/spark/commit/58
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22615
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96961/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22615
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/22615
**[Test build #96961 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96961/testReport)**
for PR 22615 at commit
[`1f16631`](https://github.com/apache/spark/commit/1
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/22635#discussion_r222905059
--- Diff: python/pyspark/accumulators.py ---
@@ -109,10 +109,14 @@
def _deserialize_accumulator(aid, zero_value, accum_param):
from pyspar
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22637
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/22637
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96974/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22637
**[Test build #96974 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96974/testReport)**
for PR 22637 at commit
[`004ee44`](https://github.com/apache/spark/commit/0
Github user ScrapCodes commented on the issue:
https://github.com/apache/spark/pull/22339
Thank you @srowen and @steveloughran.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comman
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20761
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96973/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20761
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/20761
**[Test build #96973 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96973/testReport)**
for PR 20761 at commit
[`b0e68d2`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22637
**[Test build #96974 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96974/testReport)**
for PR 22637 at commit
[`004ee44`](https://github.com/apache/spark/commit/00
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21816
**[Test build #4357 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4357/testReport)**
for PR 21816 at commit
[`5006de8`](https://github.com/apache/spark/commit/
Github user AbdealiJK commented on a diff in the pull request:
https://github.com/apache/spark/pull/22635#discussion_r222899988
--- Diff: python/pyspark/accumulators.py ---
@@ -109,10 +109,14 @@
def _deserialize_accumulator(aid, zero_value, accum_param):
from pys
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22637
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96972/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22637
Merged build finished. Test FAILed.
---
-
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/22637
**[Test build #96972 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96972/testReport)**
for PR 22637 at commit
[`14d9d1f`](https://github.com/apache/spark/commit/1
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r222897804
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnAllocatorSuite.scala
---
@@ -134,6 +166,42 @@ class YarnAllocatorSuite exte
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20761
**[Test build #96973 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96973/testReport)**
for PR 20761 at commit
[`b0e68d2`](https://github.com/apache/spark/commit/b0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22637
**[Test build #96972 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96972/testReport)**
for PR 22637 at commit
[`14d9d1f`](https://github.com/apache/spark/commit/14
Github user Fokko commented on the issue:
https://github.com/apache/spark/pull/22637
Valid points. Personally I'm a fan of explicit final, instead of implicit.
But that's a matter of taste :-)
---
-
To unsubscribe,
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22379#discussion_r222894891
--- Diff: python/pyspark/sql/functions.py ---
@@ -2664,6 +2664,28 @@ def sequence(start, stop, step=None):
_to_java_column(start), _to_jav
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r222895698
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/ClientSuite.scala
---
@@ -199,6 +200,92 @@ class ClientSuite extends SparkFunSu
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22379#discussion_r222895375
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/csvExpressions.scala
---
@@ -0,0 +1,107 @@
+/*
+ * Licensed to th
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22379#discussion_r222895573
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/csvExpressions.scala
---
@@ -0,0 +1,107 @@
+/*
+ * Licensed to th
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22637
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/22637
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96970/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22637
**[Test build #96970 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96970/testReport)**
for PR 22637 at commit
[`0155b7a`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22637
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22635
Thanks for cc'ing me. Will take a look this week.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For addi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22637
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96969/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22637
**[Test build #96969 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96969/testReport)**
for PR 22637 at commit
[`5d3d0c7`](https://github.com/apache/spark/commit/5
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r222894767
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/ClientSuite.scala
---
@@ -199,6 +200,92 @@ class ClientSuite extends SparkFunSu
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22633
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/3699/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22633
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r222894728
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/ClientSuite.scala
---
@@ -199,6 +200,92 @@ class ClientSuite extends SparkFunSu
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22633
Looks fine to me
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/22633
Looks like `lint-java` doesn't catch any style issues in my PR
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22633
**[Test build #96971 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96971/testReport)**
for PR 22633 at commit
[`999b6be`](https://github.com/apache/spark/commit/99
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22627#discussion_r222894056
--- Diff: docs/structured-streaming-programming-guide.md ---
@@ -1989,22 +2026,211 @@ head(sql("select * from aggregates"))
-#
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22627#discussion_r222894110
--- Diff: docs/structured-streaming-programming-guide.md ---
@@ -1989,22 +2026,211 @@ head(sql("select * from aggregates"))
-#
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22627#discussion_r222893790
--- Diff: docs/structured-streaming-programming-guide.md ---
@@ -1989,22 +2026,211 @@ head(sql("select * from aggregates"))
-#
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22627#discussion_r222894028
--- Diff: docs/structured-streaming-programming-guide.md ---
@@ -1989,22 +2026,211 @@ head(sql("select * from aggregates"))
-#
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22627#discussion_r222893841
--- Diff: docs/structured-streaming-programming-guide.md ---
@@ -1989,22 +2026,211 @@ head(sql("select * from aggregates"))
-#
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22627#discussion_r222893720
--- Diff: docs/structured-streaming-programming-guide.md ---
@@ -1989,22 +2026,211 @@ head(sql("select * from aggregates"))
-#
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22637
**[Test build #96970 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96970/testReport)**
for PR 22637 at commit
[`0155b7a`](https://github.com/apache/spark/commit/01
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22627#discussion_r222893572
--- Diff: docs/structured-streaming-programming-guide.md ---
@@ -1989,22 +2026,211 @@ head(sql("select * from aggregates"))
-#
Github user Fokko commented on a diff in the pull request:
https://github.com/apache/spark/pull/22637#discussion_r222893178
--- Diff:
common/network-shuffle/src/test/java/org/apache/spark/network/shuffle/ExternalShuffleIntegrationSuite.java
---
@@ -133,37 +133,38 @@ private FetchR
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22633#discussion_r222893318
--- Diff:
sql/core/src/test/java/test/org/apache/spark/sql/streaming/JavaDataStreamReaderWriterSuite.java
---
@@ -0,0 +1,89 @@
+/*
+* Licensed
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22637
**[Test build #96969 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96969/testReport)**
for PR 22637 at commit
[`5d3d0c7`](https://github.com/apache/spark/commit/5d
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22619
@HyukjinKwon Sure :-)
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22633#discussion_r222893114
--- Diff:
sql/core/src/test/java/test/org/apache/spark/sql/streaming/JavaDataStreamReaderWriterSuite.java
---
@@ -0,0 +1,89 @@
+/*
+* Licensed
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22619
Yup. Let me leave this open few more days in case.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For add
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/22635#discussion_r222892890
--- Diff: python/pyspark/accumulators.py ---
@@ -109,10 +109,14 @@
def _deserialize_accumulator(aid, zero_value, accum_param):
from pyspar
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
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/22047
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/3698/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #96968 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96968/testReport)**
for PR 22047 at commit
[`b378fff`](https://github.com/apache/spark/commit/b3
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22637
Jenkins test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22637
**[Test build #96967 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96967/testReport)**
for PR 22637 at commit
[`5d3d0c7`](https://github.com/apache/spark/commit/5d
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22637#discussion_r222892548
--- Diff:
common/network-shuffle/src/test/java/org/apache/spark/network/shuffle/ExternalShuffleIntegrationSuite.java
---
@@ -133,37 +133,38 @@ private Fetch
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22637#discussion_r222892759
--- Diff:
sql/hive-thriftserver/src/main/java/org/apache/hive/service/cli/CLIService.java
---
@@ -154,6 +154,7 @@ public synchronized void start() {
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22047
@gatorsmile First of all, thank you very much . Actually the added
aggregates weren't null filtering. I have fixed the issue and have added
additional test cases. Thank you.
---
-
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22637
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22619
@HyukjinKwon Does this look okay now ?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comm
Github user 10110346 commented on the issue:
https://github.com/apache/spark/pull/22594
@srowen Yes,I will update,thanks
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-m
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22637
mind filling PR description please?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional command
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22637
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spa
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22637
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22637
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22637
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user Fokko commented on the issue:
https://github.com/apache/spark/pull/22399
@HyukjinKwon I've opened a new PR under
https://github.com/apache/spark/pull/22637. Would be nice if you can trigger
Travis ð
---
---
GitHub user Fokko opened a pull request:
https://github.com/apache/spark/pull/22637
Spark 25408
## What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
## How was this patch tested?
(Please explain how this patch wa
Github user sandeep-katta commented on a diff in the pull request:
https://github.com/apache/spark/pull/22466#discussion_r222891761
--- Diff: python/pyspark/sql/tests.py ---
@@ -351,7 +351,7 @@ def tearDown(self):
super(SQLTests, self).tearDown()
# te
Github user sandeep-katta commented on a diff in the pull request:
https://github.com/apache/spark/pull/22466#discussion_r222891525
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -407,6 +407,7 @@ abstract class DDLSuite extends Query
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/22500
@dongjoon-hyun Is this ready to go?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r222890990
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceRequestHelper.scala
---
@@ -0,0 +1,106 @@
+/*
+ * Licensed to t
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22633#discussion_r222890961
--- Diff:
sql/core/src/test/java/test/org/apache/spark/sql/streaming/JavaDataStreamReaderWriterSuite.java
---
@@ -0,0 +1,89 @@
+/*
+* License
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r222890547
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/TestYarnResourceRequestHelper.scala
---
@@ -0,0 +1,92 @@
+/*
+ * Licens
Github user AbdealiJK commented on a diff in the pull request:
https://github.com/apache/spark/pull/22635#discussion_r222890103
--- Diff: python/pyspark/accumulators.py ---
@@ -109,10 +109,14 @@
def _deserialize_accumulator(aid, zero_value, accum_param):
from pys
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22399
Let me trigger it in the next PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r222889602
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/ResourceRequestValidatorSuite.scala
---
@@ -0,0 +1,132 @@
+/*
+ * Licen
Github user Fokko commented on the issue:
https://github.com/apache/spark/pull/22399
The tests passed earlier, how would it be possible that it would fail on
master?
---
-
To unsubscribe, e-mail: reviews-unsubscr...
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r222889144
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/TestYarnResourceRequestHelper.scala
---
@@ -0,0 +1,92 @@
+/*
+ * Licens
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22399
@Fokko, Let's follow the PR format next time BTW, for instance, "How was
this patch tested?"
---
-
To unsubscribe, e-mail: r
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22399
It wasn't a merge conflict. I failed to notice this had not actually been
tested. To check locally you'd have to make sure you enable more profiles like
Hive to build some of the code that changed he
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r222888986
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/TestYarnResourceRequestHelper.scala
---
@@ -0,0 +1,92 @@
+/*
+ * Licens
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22633#discussion_r222888557
--- Diff:
sql/core/src/test/java/test/org/apache/spark/sql/streaming/JavaDataStreamReaderWriterSuite.java
---
@@ -0,0 +1,89 @@
+/*
+* License
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r222888518
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/ClientSuite.scala
---
@@ -199,6 +200,92 @@ class ClientSuite extends SparkFunSu
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r222888391
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/ClientSuite.scala
---
@@ -199,6 +200,92 @@ class ClientSuite extends SparkFunSu
Github user Fokko commented on the issue:
https://github.com/apache/spark/pull/22399
Thanks @srowen for pointing out the errors. Weird that it did not come up
as a merge conflict. Let me open a new PR.
---
-
To unsu
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r222887537
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/ClientSuite.scala
---
@@ -35,13 +36,13 @@ import org.apache.hadoop.yarn.conf.Ya
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r222887404
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceRequestValidator.scala
---
@@ -0,0 +1,70 @@
+/*
+ * Licensed to
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r222887435
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceRequestValidator.scala
---
@@ -0,0 +1,70 @@
+/*
+ * Licensed to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22634
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/22634
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96959/
Test PASSed.
---
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r222886705
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceRequestValidator.scala
---
@@ -0,0 +1,70 @@
+/*
+ * Licensed to
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r222886619
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceRequestValidator.scala
---
@@ -0,0 +1,70 @@
+/*
+ * Licensed to
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22634
**[Test build #96959 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96959/testReport)**
for PR 22634 at commit
[`0cdd4e5`](https://github.com/apache/spark/commit/0
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r222886571
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceRequestHelper.scala
---
@@ -0,0 +1,106 @@
+/*
+ * Licensed to t
1 - 100 of 532 matches
Mail list logo