Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/14079#discussion_r70005830
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala ---
@@ -236,29 +246,43 @@ private[spark] class TaskSchedulerImpl(
*
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14098
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14098
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61939/
Test FAILed.
---
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/14079#discussion_r70007341
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/BlacklistTracker.scala ---
@@ -0,0 +1,320 @@
+/*
+ * Licensed to the Apache Software
Github user keypointt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14082#discussion_r70007444
--- Diff: examples/src/main/r/RSparkSQLExample.R ---
@@ -0,0 +1,175 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/14079#discussion_r70008186
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/TaskSchedulerImplSuite.scala ---
@@ -281,6 +300,174 @@ class TaskSchedulerImplSuite extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14098
**[Test build #61940 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61940/consoleFull)**
for PR 14098 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14098
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14096
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61941/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14098
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61940/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14096
**[Test build #61941 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61941/consoleFull)**
for PR 14096 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14096
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14094
**[Test build #61927 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61927/consoleFull)**
for PR 14094 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14094
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
GitHub user dongjoon-hyun opened a pull request:
https://github.com/apache/spark/pull/14096
[SPARK-16425][R] `describe()` should consider numeric/string-type columns
## What changes were proposed in this pull request?
This PR prevents ERRORs when `summary(df)` is called for
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14096
Currently, Scala/Python already do column-type checking for this.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/13984
LGTM. Merging this to master and branch-2.0
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/13984#discussion_r69997940
--- Diff: R/pkg/R/SQLContext.R ---
@@ -744,6 +747,9 @@ read.df.default <- function(path = NULL, source = NULL,
schema = NULL, ...) {
if
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14095
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61929/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14095
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14095
**[Test build #61929 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61929/consoleFull)**
for PR 14095 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14049
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14094
**[Test build #61928 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61928/consoleFull)**
for PR 14094 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14049
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61932/
Test PASSed.
---
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14090#discussion_r7362
--- Diff: docs/sparkr.md ---
@@ -306,6 +306,64 @@ head(ldf, 3)
{% endhighlight %}
+ Run a given function on a large dataset
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/14090
LGTM except for comment on "schema matching".
Also I wonder if we should rephrase "can only be used if the output of UDF
run on all the partitions can fit in driver memory" - it seems not as
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14096#discussion_r70003846
--- Diff: R/pkg/inst/tests/testthat/test_sparkSQL.R ---
@@ -1804,11 +1804,11 @@ test_that("describe() and summarize() on a
DataFrame", {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14096
**[Test build #61935 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61935/consoleFull)**
for PR 14096 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14094
**[Test build #61936 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61936/consoleFull)**
for PR 14094 at commit
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/14079#discussion_r70005147
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/BlacklistTracker.scala ---
@@ -0,0 +1,320 @@
+/*
+ * Licensed to the Apache Software
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/14079#discussion_r70005525
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/BlacklistTracker.scala ---
@@ -0,0 +1,320 @@
+/*
+ * Licensed to the Apache Software
Github user wangmiao1981 commented on the issue:
https://github.com/apache/spark/pull/14098
@liancheng Can you review it? Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user sharkdtu commented on the issue:
https://github.com/apache/spark/pull/14088
@tgravescs fixed the description and style
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/14096#discussion_r70006844
--- Diff: R/pkg/inst/tests/testthat/test_sparkSQL.R ---
@@ -1804,11 +1804,11 @@ test_that("describe() and summarize() on a
DataFrame", {
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/14079#discussion_r70006954
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/BlacklistTracker.scala ---
@@ -0,0 +1,320 @@
+/*
+ * Licensed to the Apache Software
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14082#discussion_r70008001
--- Diff: examples/src/main/r/RSparkSQLExample.R ---
@@ -0,0 +1,175 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14096
**[Test build #61938 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61938/consoleFull)**
for PR 14096 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14096
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/14079#discussion_r70008649
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/BlacklistTracker.scala ---
@@ -0,0 +1,320 @@
+/*
+ * Licensed to the Apache Software
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14082#discussion_r70009456
--- Diff: examples/src/main/r/RSparkSQLExample.R ---
@@ -0,0 +1,175 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
GitHub user ericl opened a pull request:
https://github.com/apache/spark/pull/14099
[SPARK-16432] Empty blocks fail to serialize due to assert in
ChunkedByteBuffer
## What changes were proposed in this pull request?
It's possible to also change the callers to not pass in
Github user keypointt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14082#discussion_r70011061
--- Diff: examples/src/main/r/RSparkSQLExample.R ---
@@ -0,0 +1,175 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/13680
LGTM except some minor comments, it's pretty close! One easy-to-ignore
comment: https://github.com/apache/spark/pull/13680/files#r69849567
---
If your project is set up for it, you can reply to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14096
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61942/
Test PASSed.
---
Github user keypointt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14082#discussion_r70011796
--- Diff: examples/src/main/r/RSparkSQLExample.R ---
@@ -0,0 +1,175 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14082#discussion_r70012365
--- Diff: examples/src/main/r/RSparkSQLExample.R ---
@@ -0,0 +1,175 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14096
Thank you!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14071#discussion_r70012893
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala
---
@@ -403,17 +400,18 @@ object
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14101
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14099
**[Test build #61944 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61944/consoleFull)**
for PR 14099 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14099
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61944/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14099
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/13701
@gatorsmile This is the benchmark results. No significant difference.
Before this patch:
Java HotSpot(TM) 64-Bit Server VM 1.8.0_71-b15 on Linux
3.19.0-25-generic
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13991
**[Test build #61947 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61947/consoleFull)**
for PR 13991 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13991
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13991
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61947/
Test PASSed.
---
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/14008#discussion_r70020485
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -652,6 +654,152 @@ case class StringRPad(str:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13991
**[Test build #61953 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61953/consoleFull)**
for PR 13991 at commit
Github user petermaxlee commented on the issue:
https://github.com/apache/spark/pull/14103
cc @cloud-fan
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13890
**[Test build #61950 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61950/consoleFull)**
for PR 13890 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13890
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61950/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13890
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
GitHub user petermaxlee opened a pull request:
https://github.com/apache/spark/pull/14103
[SPARK-16436][SQL] checkEvaluation support NaN and RuntimeReplaceable
## What changes were proposed in this pull request?
This small patch modifies ExpressionEvalHelper. checkEvaluation to
GitHub user zsxwing opened a pull request:
https://github.com/apache/spark/pull/14100
[SPARK-16433][SQL]Improve StreamingQuery.explain when no data arrives
## What changes were proposed in this pull request?
Display `No physical plan. Waiting for data.` instead of `N/A`
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14096
LGTM. Thanks @dongjoon-hyun -- Merging this to master, branch-2.0
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14096
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14096
Hi, @shivaram .
Now, it's ready for review again.
Please let me know if there is something to do more.
Thank you!
---
If your project is set up for it, you can reply to this email
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14082
**[Test build #61946 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61946/consoleFull)**
for PR 14082 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14100
**[Test build #61945 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61945/consoleFull)**
for PR 14100 at commit
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/13991
For this one I think we should consider supporting only foldable literals
for the path component.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13991
**[Test build #61947 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61947/consoleFull)**
for PR 13991 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14102
**[Test build #61948 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61948/consoleFull)**
for PR 14102 at commit
Github user husseinhazimeh commented on the issue:
https://github.com/apache/spark/pull/14101
@mengxr @sethah can you review this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14102
**[Test build #61948 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61948/consoleFull)**
for PR 14102 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14102
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14102
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61948/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14102
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61949/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14102
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14008#discussion_r70020929
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -652,6 +654,152 @@ case class
Github user janplus commented on a diff in the pull request:
https://github.com/apache/spark/pull/14008#discussion_r70021325
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -652,6 +654,152 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14083
**[Test build #61952 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61952/consoleFull)**
for PR 14083 at commit
Github user petermaxlee commented on the issue:
https://github.com/apache/spark/pull/13991
I pushed a new change to this. We now have better error messages and test
coverage for those. These expressions also now require foldable paths.
I also changed the test values to make
Github user petermaxlee commented on the issue:
https://github.com/apache/spark/pull/13969
Ping!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14096
**[Test build #61942 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61942/consoleFull)**
for PR 14096 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14094
**[Test build #61936 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61936/consoleFull)**
for PR 14094 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14094
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14094
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61936/
Test PASSed.
---
GitHub user HyukjinKwon opened a pull request:
https://github.com/apache/spark/pull/14102
[SPARK-16434][SQL][WIP] Avoid record-per type dispatch in JSON when reading
## What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/13991
Also - rather than having concrete implementations for all of these, why
don't we use RuntimeReplaceable?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14102
**[Test build #61949 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61949/consoleFull)**
for PR 14102 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14028
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61943/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14028
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14071#discussion_r70017144
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/MetastoreRelation.scala ---
@@ -162,25 +147,28 @@ private[hive] case class MetastoreRelation(
Github user janplus commented on a diff in the pull request:
https://github.com/apache/spark/pull/14008#discussion_r70018330
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -652,6 +654,145 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14100
**[Test build #61945 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61945/consoleFull)**
for PR 14100 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14100
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61945/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14100
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/14065#discussion_r70020146
--- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala ---
@@ -390,8 +390,9 @@ private[spark] class Client(
// Upload Spark and
Github user janplus commented on the issue:
https://github.com/apache/spark/pull/14008
cc @cloud-fan @rxin @liancheng
I did optimization for Literal `part`, so we don't need to check for every
row. But since we may not assume in all circumstances the `part` is Literal, I
keep
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/14100
cc @tdas
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
401 - 500 of 591 matches
Mail list logo
