Github user srowen commented on the issue:
https://github.com/apache/spark/pull/18874
Going to 0 executors is not a bug, if you set the min to 0. A deadlock is a
bug. But, nothing in the JIRA or here suggests there's a deadlock -- what do
you mean?
---
If your project is set up
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/18787#discussion_r132246475
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ColumnarBatch.java
---
@@ -65,15 +65,35 @@
final Row row;
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/16774
Thanks for taking a look @WeichenXu123!
---
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 tgravescs commented on the issue:
https://github.com/apache/spark/pull/18874
I'm saying you have a stage running that has > 0 tasks to run. If dynamic
allocation has already got all the executors it originally thought it needed
and they all idle timeout then you have 0
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/18893
Though I think these should have been separate PRs both these changes LGTM,
I'm not sure why the original choice was to only display the pagination nabber
at the top and though I'm not usually a
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18630#discussion_r132262219
--- Diff: core/src/main/scala/org/apache/spark/deploy/DependencyUtils.scala
---
@@ -0,0 +1,101 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18630#discussion_r132262064
--- Diff: core/src/main/scala/org/apache/spark/deploy/DependencyUtils.scala
---
@@ -0,0 +1,101 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18630#discussion_r132262101
--- Diff: core/src/main/scala/org/apache/spark/deploy/DependencyUtils.scala
---
@@ -0,0 +1,101 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18630#discussion_r132262287
--- Diff: core/src/main/scala/org/apache/spark/deploy/DependencyUtils.scala
---
@@ -0,0 +1,101 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18630#discussion_r132262559
--- Diff:
core/src/main/scala/org/apache/spark/deploy/worker/DriverWrapper.scala ---
@@ -66,4 +70,24 @@ object DriverWrapper {
System.exit(-1)
Github user sayeaud commented on the issue:
https://github.com/apache/spark/pull/17455
Would love to get this in for the next release, currently I'm using a
variant of https://github.com/aseigneurin/spark-ui-proxy to work around this
issue...
---
If your project is set up for it,
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/18640
Retest this please.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18648
**[Test build #80458 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80458/testReport)**
for PR 18648 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18648
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80458/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18648
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/18648
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80460/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18648
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/18883
---
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 feature is
Github user peay commented on the issue:
https://github.com/apache/spark/pull/18752
Great, thanks @JoshRosen!
---
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
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/18787#discussion_r132252558
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ColumnarBatch.java
---
@@ -65,15 +65,35 @@
final Row row;
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18849
**[Test build #80463 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80463/testReport)**
for PR 18849 at commit
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/18874
I suggest you go understand the code.
I've already explained this multiple times. You get 0 executors by there
being delays when an executors doesn't have a task scheduled. say you
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/17455
#18499 was just merged, you'll need to update this to match the new code if
you hope to get reinvigorate interest
---
If your project is set up for it, you can reply to this email and have your
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/18640
@rxin . Could you make some decision for this PR? Do we need to put this
into `sql/hive` still for some reasons?
---
If your project is set up for it, you can reply to this email and have
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/1
**[Test build #80465 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80465/testReport)**
for PR 1 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18640
**[Test build #80466 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80466/testReport)**
for PR 18640 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18648
**[Test build #80460 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80460/testReport)**
for PR 18648 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18896
**[Test build #80462 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80462/testReport)**
for PR 18896 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18896
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/18896
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80462/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18820
**[Test build #80459 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80459/testReport)**
for PR 18820 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18820
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/18820
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80459/
Test FAILed.
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18820
retest this please
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18820
**[Test build #80464 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80464/testReport)**
for PR 18820 at commit
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/18630#discussion_r132274194
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -300,28 +298,14 @@ object SparkSubmit extends CommandLineUtils {
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17849#discussion_r132360643
--- Diff: python/pyspark/ml/classification.py ---
@@ -1325,7 +1325,7 @@ def __init__(self, featuresCol="features",
labelCol="label", predictionCol="pred
Github user debugger87 commented on the issue:
https://github.com/apache/spark/pull/18900
@cloud-fan could you please help me to review this PR?
---
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
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18810#discussion_r132360895
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -356,6 +356,19 @@ class
GitHub user debugger87 opened a pull request:
https://github.com/apache/spark/pull/18900
[SPARK-21687][SQL] Spark SQL should set createTime for Hive partition
## What changes were proposed in this pull request?
Set createTime for every hive partition created in Spark SQL,
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18810#discussion_r132363994
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -356,6 +356,19 @@ class
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18810#discussion_r132367041
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -572,6 +572,14 @@ object SQLConf {
"disable logging or
Github user eatoncys commented on a diff in the pull request:
https://github.com/apache/spark/pull/18810#discussion_r132368484
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala
---
@@ -370,6 +370,14 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18810
**[Test build #80470 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80470/testReport)**
for PR 18810 at commit
Github user phatak-dev commented on the issue:
https://github.com/apache/spark/pull/17972
@MLnick Any updates on 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 project does not have this feature
enabled
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18810#discussion_r132359678
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala
---
@@ -370,6 +370,15 @@ case class
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17849#discussion_r132361043
--- Diff: python/pyspark/ml/tests.py ---
@@ -1572,7 +1588,8 @@ def test_java_params(self):
for name, cls in inspect.getmembers(module,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18810
**[Test build #80472 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80472/testReport)**
for PR 18810 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18900
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 viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18810#discussion_r132367400
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -572,6 +572,14 @@ object SQLConf {
"disable logging or
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18810
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/18810
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80470/
Test PASSed.
---
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17849#discussion_r132359369
--- Diff: python/pyspark/ml/wrapper.py ---
@@ -144,7 +158,9 @@ def _transfer_params_from_java(self):
if self._java_obj.hasParam(param.name):
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/18865#discussion_r132363283
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JsonFileFormat.scala
---
@@ -114,7 +114,16 @@ class JsonFileFormat
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18865#discussion_r132364612
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JsonFileFormat.scala
---
@@ -114,7 +114,16 @@ class JsonFileFormat
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18810#discussion_r132366187
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala
---
@@ -370,6 +370,14 @@ case class
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18810
Btw, can you change `[sql]` to `[SQL]` in title?
---
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 viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17849#discussion_r132358656
--- Diff: python/pyspark/ml/wrapper.py ---
@@ -263,7 +284,8 @@ def _fit_java(self, dataset):
def _fit(self, dataset):
java_model
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18810
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80472/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18810
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 a diff in the pull request:
https://github.com/apache/spark/pull/18865#discussion_r132361425
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JsonFileFormat.scala
---
@@ -114,7 +114,16 @@ class JsonFileFormat
Github user eatoncys commented on a diff in the pull request:
https://github.com/apache/spark/pull/18810#discussion_r132365401
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -356,6 +356,19 @@ class CodegenContext
Github user eatoncys commented on a diff in the pull request:
https://github.com/apache/spark/pull/18810#discussion_r132365436
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -356,6 +356,19 @@ class CodegenContext
Github user eatoncys commented on a diff in the pull request:
https://github.com/apache/spark/pull/18810#discussion_r132365359
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -572,6 +572,13 @@ object SQLConf {
"disable logging or
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18810
**[Test build #80475 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80475/testReport)**
for PR 18810 at commit
Github user zhengruifeng closed the pull request at:
https://github.com/apache/spark/pull/17995
---
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 feature
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18810#discussion_r132366896
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeFormatter.scala
---
@@ -89,6 +89,14 @@ object CodeFormatter {
Github user eatoncys commented on a diff in the pull request:
https://github.com/apache/spark/pull/18810#discussion_r132368646
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -572,6 +572,14 @@ object SQLConf {
"disable logging or
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/17849
Sorry, let me try and take a look tomorrow.
---
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 viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17849#discussion_r132360069
--- Diff: python/pyspark/ml/tests.py ---
@@ -417,6 +417,54 @@ def test_logistic_regression_check_thresholds(self):
LogisticRegression,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18810
**[Test build #80471 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80471/testReport)**
for PR 18810 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18810
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80471/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18810
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 gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18810#discussion_r132360710
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -572,6 +572,13 @@ object SQLConf {
"disable logging
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18865#discussion_r132363687
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JsonFileFormat.scala
---
@@ -114,7 +114,16 @@ class JsonFileFormat
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18869
@viirya Could you hold this PR at first? There is a related PR
https://github.com/apache/spark/pull/18818 that is changing the underlying
semantics. We also need to update the description after
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18421#discussion_r132099286
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -1033,25 +998,126 @@ private[spark] class
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18499
@aosagie do we still need the previous fix for SPARK-21176?
---
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
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/18797
Merged into master and branch-2.2. Thanks for all.
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18797
---
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 feature is
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18820#discussion_r132104291
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameNaFunctionsSuite.scala ---
@@ -261,5 +261,37 @@ class DataFrameNaFunctionsSuite extends
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/17373
I think this will change the output of `summary` on `spark.mlp` in R right?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18880
**[Test build #80444 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80444/testReport)**
for PR 18880 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18648
**[Test build #80442 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80442/testReport)**
for PR 18648 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18709
**[Test build #80445 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80445/testReport)**
for PR 18709 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18820
LGTM except a few minor comments.
---
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/18810#discussion_r132105955
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -356,6 +356,18 @@ class
Github user lvdongr commented on the issue:
https://github.com/apache/spark/pull/18756
You can see this picture,my table has three columns,and I insert only two
columns, then the last column is null. @maropu @gatorsmile
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18810#discussion_r132107421
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/AggregateBenchmark.scala
---
@@ -301,6 +301,61 @@ class AggregateBenchmark
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18849#discussion_r132107646
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/Hive_2_1_DDLSuite.scala
---
@@ -0,0 +1,131 @@
+/*
+ * Licensed to the
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/18879
I'm not convinced this deserves a separated PR because it's neither bugfix
nor any improvement. Further more, the method `Utils.localHostName()` is used
in many other places, we can't change
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/18891
Close this @xiaosannimei
---
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
Github user IngoSchuster commented on the issue:
https://github.com/apache/spark/pull/18499
Yes I believe it still makes sense to limit the number of selector threads.
Jetty's default is tuned for a real web server and by default it creates
(number of cpus / 2) selector threads.
Github user yaooqinn commented on the issue:
https://github.com/apache/spark/pull/18648
retest this please
---
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/18869
**[Test build #80435 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80435/testReport)**
for PR 18869 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18869
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80435/
Test PASSed.
---
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18869
@gatorsmile Sure. Let's wait after #18818. I'll look at it too.
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18869
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 the issue:
https://github.com/apache/spark/pull/18818
Also add `NULL` in the test case?
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/18499
---
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 feature is
1 - 100 of 392 matches
Mail list logo