Github user srowen commented on the issue:
https://github.com/apache/spark/pull/21179
Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21301
**[Test build #4177 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4177/testReport)**
for PR 21301 at commit
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/21315
LGTM. Lets wait till #21314 is merged ?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21301
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90556/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21301
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21301
**[Test build #90556 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90556/testReport)**
for PR 21301 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21092
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3088/
---
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/21314#discussion_r187815762
--- Diff: R/pkg/R/client.R ---
@@ -82,7 +82,7 @@ checkJavaVersion <- function() {
})
javaVersionFilter <- Filter(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21296
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21296
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90555/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21092
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3088/
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21296
**[Test build #90555 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90555/testReport)**
for PR 21296 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21092
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/3182/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21092
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21208
**[Test build #90562 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90562/testReport)**
for PR 21208 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21316
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21315
@shivaram
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21316
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user sohama4 opened a pull request:
https://github.com/apache/spark/pull/21316
[SPARK-20538][SQL] Wrap Dataset.reduce with withNewExecutionId.
## What changes were proposed in this pull request?
Wrap Dataset.reduce with `withNewExecutionId`.
You can merge this pull
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/21314#discussion_r187815502
--- Diff: R/pkg/R/client.R ---
@@ -82,7 +82,7 @@ checkJavaVersion <- function() {
})
javaVersionFilter <- Filter(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21092
**[Test build #90561 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90561/testReport)**
for PR 21092 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21313
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21313
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90560/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21313
**[Test build #90560 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90560/testReport)**
for PR 21313 at commit
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/21314#discussion_r187815145
--- Diff: R/pkg/R/client.R ---
@@ -82,7 +82,7 @@ checkJavaVersion <- function() {
})
javaVersionFilter <- Filter(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21315
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90559/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21315
**[Test build #90559 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90559/testReport)**
for PR 21315 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21315
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21313
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21313
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/3181/
Github user huaxingao commented on the issue:
https://github.com/apache/spark/pull/21313
There are still quite a lot of the SQL functions to be added in R. We can
bundle several of the functions together in one PR, but I guess it's too much
work to add all of them in one PR.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21313
**[Test build #90560 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90560/testReport)**
for PR 21313 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21314
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90558/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21314
**[Test build #90558 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90558/testReport)**
for PR 21314 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21314
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user huaxingao commented on a diff in the pull request:
https://github.com/apache/spark/pull/21313#discussion_r187814160
--- Diff: R/pkg/R/functions.R ---
@@ -3006,6 +3008,28 @@ setMethod("array_contains",
column(jc)
})
+#' @details
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21315
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21315
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/3180/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21315
**[Test build #90559 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90559/testReport)**
for PR 21315 at commit
GitHub user felixcheung opened a pull request:
https://github.com/apache/spark/pull/21315
[SPARK-23780][R] Failed to use googleVis library with new SparkR
## What changes were proposed in this pull request?
change generic to get it to work with googleVis
also fix lintr
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21315
the main change is
https://github.com/apache/spark/compare/master...felixcheung:googvis?expand=1#diff-8e3d61ff66c9ffcd6ffb7a8eedc08409
---
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/21295
@rdblue . If this is a general bug for previous Spark 2.3 and below, we
should include this fix into 2.3.1 with a proper reproducible test case with
Parquet 1.8.3.
cc @vanzin because
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/21295#discussion_r187813544
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/SpecificParquetRecordReaderBase.java
---
@@ -147,7 +147,8 @@
Github user mn-mikke commented on a diff in the pull request:
https://github.com/apache/spark/pull/21236#discussion_r187813418
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -118,6 +119,161 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21314
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21314
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/3179/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21314
**[Test build #90558 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90558/testReport)**
for PR 21314 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21314
@vanzin FYI - need to fix this before release
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21314
@shivaram
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
GitHub user felixcheung opened a pull request:
https://github.com/apache/spark/pull/21314
[SPARK-24263] SparkR java check breaks with openjdk
## What changes were proposed in this pull request?
Change text to grep for.
## How was this patch tested?
manual
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21236
**[Test build #90557 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90557/testReport)**
for PR 21236 at commit
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/21295#discussion_r187813149
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetQuerySuite.scala
---
@@ -879,6 +879,18 @@ class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21301
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21301
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/3178/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21301
**[Test build #90556 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90556/testReport)**
for PR 21301 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/21301
Retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21304
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/21304
LGTM, @robinske & I are hanging out in the United lounge so were just going
to merge this now.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21301
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90554/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21301
**[Test build #90554 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90554/testReport)**
for PR 21301 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21301
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user ifilonenko commented on a diff in the pull request:
https://github.com/apache/spark/pull/21092#discussion_r187811483
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/BasicExecutorFeatureStep.scala
---
@@ -54,7 +54,8 @@
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/21313#discussion_r187810848
--- Diff: R/pkg/R/functions.R ---
@@ -3006,6 +3008,28 @@ setMethod("array_contains",
column(jc)
})
+#'
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21179
**[Test build #4174 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4174/testReport)**
for PR 21179 at commit
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21296#discussion_r187810542
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/csv/CSVSuite.scala
---
@@ -1322,4 +1322,31 @@ class CSVSuite extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21296
**[Test build #90555 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90555/testReport)**
for PR 21296 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19293
**[Test build #4176 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4176/testReport)**
for PR 19293 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18903
**[Test build #4175 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4175/testReport)**
for PR 18903 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/21162
Please close this PR. That is the rule that actual does shading of Guava,
and we most definitely want to keep it.
---
-
To
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/21295#discussion_r187809226
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetQuerySuite.scala
---
@@ -879,6 +879,18 @@ class
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/21295#discussion_r187809171
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/SpecificParquetRecordReaderBase.java
---
@@ -147,7 +147,8 @@ public
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
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/90553/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20761
**[Test build #90553 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90553/testReport)**
for PR 20761 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21301
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21301
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/3177/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21301
**[Test build #90554 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90554/testReport)**
for PR 21301 at commit
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21301#discussion_r187808776
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/reader/partitioning/Distribution.java
---
@@ -22,7 +22,8 @@
/**
* An
Github user szyszy commented on the issue:
https://github.com/apache/spark/pull/20761
@vanzin : Reply to your main comment from above:
For the documentation: Please advise which documentation(s) should I update
and where!
For your question about the case when user sets
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r187808559
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/ClientSuite.scala
---
@@ -35,17 +36,23 @@ import
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r187808552
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeValidator.scala
---
@@ -0,0 +1,162 @@
+/*
+ * Licensed to
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r187808565
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/ClientSuite.scala
---
@@ -199,6 +206,91 @@ class ClientSuite extends
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r187808534
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeValidator.scala
---
@@ -0,0 +1,162 @@
+/*
+ * Licensed to
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r187808533
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeValidator.scala
---
@@ -0,0 +1,162 @@
+/*
+ * Licensed to
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r187808512
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeValidator.scala
---
@@ -0,0 +1,162 @@
+/*
+ * Licensed to
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r187808526
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeValidator.scala
---
@@ -0,0 +1,162 @@
+/*
+ * Licensed to
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r187808516
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeHelper.scala
---
@@ -0,0 +1,128 @@
+/*
+ * Licensed to the
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r187808508
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeValidator.scala
---
@@ -0,0 +1,162 @@
+/*
+ * Licensed to
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r187808504
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeHelper.scala
---
@@ -0,0 +1,128 @@
+/*
+ * Licensed to the
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r187808501
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeHelper.scala
---
@@ -0,0 +1,128 @@
+/*
+ * Licensed to the
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r187808492
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeHelper.scala
---
@@ -0,0 +1,128 @@
+/*
+ * Licensed to the
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r187808495
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeHelper.scala
---
@@ -0,0 +1,128 @@
+/*
+ * Licensed to the
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r187808488
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeHelper.scala
---
@@ -0,0 +1,128 @@
+/*
+ * Licensed to the
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r187808479
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeHelper.scala
---
@@ -0,0 +1,128 @@
+/*
+ * Licensed to the
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r187808482
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeHelper.scala
---
@@ -0,0 +1,128 @@
+/*
+ * Licensed to the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20761
**[Test build #90553 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90553/testReport)**
for PR 20761 at commit
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r187808470
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeHelper.scala
---
@@ -0,0 +1,128 @@
+/*
+ * Licensed to the
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r187808457
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala
---
@@ -86,6 +86,13 @@ private[spark] class Client(
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r187808466
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeHelper.scala
---
@@ -0,0 +1,128 @@
+/*
+ * Licensed to the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21313
**[Test build #90552 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90552/testReport)**
for PR 21313 at commit
101 - 200 of 246 matches
Mail list logo
