Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20230
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85977/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20230
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/20230
**[Test build #85977 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85977/testReport)**
for PR 20230 at commit
[`cc3321c`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20239
**[Test build #85989 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85989/testReport)**
for PR 20239 at commit
[`0e59098`](https://github.com/apache/spark/commit/0e
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/20239
@BryanCutler I think this comes up in the Arrow sync yesterday
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.
GitHub user icexelloss opened a pull request:
https://github.com/apache/spark/pull/20239
[SPARK-23047][PYTHON][SQL] Change MapVector to NullableMapVector in
ArrowColumnVector
## What changes were proposed in this pull request?
This PR changes usage of `MapVector` in Spark codeba
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/20239
cc @BryanCutler @ueshin
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: r
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/20203#discussion_r161070422
--- Diff:
core/src/main/scala/org/apache/spark/status/AppStatusListener.scala ---
@@ -211,6 +211,11 @@ private[spark] class AppStatusListener(
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/20216
Hiding Tables like this has been done elsewhere in the UI (can't remember
where right now though). In that case there's a triangle marker next to the
Header that shows that it's collapsable and it
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20236
Yeah, arbitrary name is fine. I tried it yesterday and the script just
yelled at me because it was not an integer. (Filtering out "Apache Spark" is
bonus.)
---
Github user squito commented on the issue:
https://github.com/apache/spark/pull/20236
@vanzin what exactly are you looking for? The one thing which would be
easy is letting you write in an arbitrary jira id (no name searching or
anything), that sound OK?
I guess this bug isn
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/20163
+1
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apac
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20072
**[Test build #85988 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85988/testReport)**
for PR 20072 at commit
[`c584c61`](https://github.com/apache/spark/commit/c5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20233
**[Test build #4035 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4035/testReport)**
for PR 20233 at commit
[`e9cf606`](https://github.com/apache/spark/commit/
Github user jose-torres commented on the issue:
https://github.com/apache/spark/pull/20225
The hang in test build #85939 was a test issue in ContinuousStressSuite,
which I could reproduce locally by bumping up the rows per second.
When the rate of incoming data is too high, th
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20228
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85976/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20225
**[Test build #85987 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85987/testReport)**
for PR 20225 at commit
[`3b19fcb`](https://github.com/apache/spark/commit/3b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20228
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/20228
**[Test build #85976 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85976/testReport)**
for PR 20228 at commit
[`fffd109`](https://github.com/apache/spark/commit/f
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/20225#discussion_r161056545
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/continuous/ContinuousSuite.scala
---
@@ -219,6 +199,37 @@ class ContinuousSuite extend
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20236
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/20236
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85973/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20236
**[Test build #85973 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85973/testReport)**
for PR 20236 at commit
[`8c4cc61`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20229
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85982/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20229
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/20223
**[Test build #85986 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85986/testReport)**
for PR 20223 at commit
[`8d52338`](https://github.com/apache/spark/commit/8d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20238
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85981/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20219
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85975/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20238
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/20219
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/20229
**[Test build #85982 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85982/testReport)**
for PR 20229 at commit
[`73fb60c`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20238
**[Test build #85981 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85981/testReport)**
for PR 20238 at commit
[`ff4d1cf`](https://github.com/apache/spark/commit/f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20219
**[Test build #85975 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85975/testReport)**
for PR 20219 at commit
[`27f0e14`](https://github.com/apache/spark/commit/2
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20236
Since you're here, would you like to try at add SPARK-23031 to this change?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spar
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20203#discussion_r161045636
--- Diff:
core/src/main/scala/org/apache/spark/status/AppStatusListener.scala ---
@@ -223,6 +228,15 @@ private[spark] class AppStatusListener(
updat
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/20225#discussion_r161045063
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/continuous/ContinuousSuite.scala
---
@@ -219,6 +199,37 @@ class ContinuousSuite extends Conti
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20096
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20214
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/20214
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85972/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20214
**[Test build #85972 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85972/testReport)**
for PR 20214 at commit
[`66b06c3`](https://github.com/apache/spark/commit/6
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20146#discussion_r161039325
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/feature/StringIndexerSuite.scala ---
@@ -33,12 +33,38 @@ class StringIndexerSuite
test(
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20146#discussion_r161040131
--- Diff: mllib/src/main/scala/org/apache/spark/ml/param/params.scala ---
@@ -249,6 +249,16 @@ object ParamValidators {
def arrayLengthGt[T](lowe
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20238
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85979/
Test PASSed.
---
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20146#discussion_r161040537
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/feature/StringIndexerSuite.scala ---
@@ -331,4 +357,51 @@ class StringIndexerSuite
val dfWi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20238
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/20238
**[Test build #85979 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85979/testReport)**
for PR 20238 at commit
[`09f0d60`](https://github.com/apache/spark/commit/0
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20192
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20192
Merging to master / 2.3.
---
-
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/19054
**[Test build #85985 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85985/testReport)**
for PR 19054 at commit
[`c689ff1`](https://github.com/apache/spark/commit/c6
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20229
**[Test build #85978 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85978/testReport)**
for PR 20229 at commit
[`17854de`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20229
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85978/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20229
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/20226#discussion_r161036084
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/HiveTableScanExec.scala
---
@@ -62,6 +62,8 @@ case class HiveTableScanExec(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20223
**[Test build #85984 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85984/testReport)**
for PR 20223 at commit
[`9c7d2d3`](https://github.com/apache/spark/commit/9c
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20153#discussion_r161034623
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2ScanExec.scala
---
@@ -37,40 +35,58 @@ import org.apache.sp
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20072
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/20072
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85970/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20208
**[Test build #85983 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85983/testReport)**
for PR 20208 at commit
[`499801e`](https://github.com/apache/spark/commit/49
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20229
**[Test build #85982 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85982/testReport)**
for PR 20229 at commit
[`73fb60c`](https://github.com/apache/spark/commit/73
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20238
**[Test build #85981 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85981/testReport)**
for PR 20238 at commit
[`ff4d1cf`](https://github.com/apache/spark/commit/ff
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20072
**[Test build #85970 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85970/testReport)**
for PR 20072 at commit
[`6fe8589`](https://github.com/apache/spark/commit/6
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20208
Retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20222
**[Test build #85980 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85980/testReport)**
for PR 20222 at commit
[`5eded03`](https://github.com/apache/spark/commit/5e
Github user shaneknapp commented on the issue:
https://github.com/apache/spark/pull/20222
confused:
https://user-images.githubusercontent.com/1606572/34838548-533a1790-f6b3-11e7-9fbc-a035a782ccfb.png";>
anyways, i will change the timeout. again.
---
-
Github user shaneknapp commented on the issue:
https://github.com/apache/spark/pull/20222
test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20220
Yeah I figured it's minor enough we can go ahead and not bother
back-porting to 2.3. I don't mind anyone else backporting.
---
-
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20238
**[Test build #85979 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85979/testReport)**
for PR 20238 at commit
[`09f0d60`](https://github.com/apache/spark/commit/09
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20220
Hi, @srowen .
Is this the first Spark 2.4-only commit?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.o
GitHub user MrBago opened a pull request:
https://github.com/apache/spark/pull/20238
Have RFormula include VectorSizeHint in pipeline
## What changes were proposed in this pull request?
Including VectorSizeHint in RFormula piplelines will allow them to be
applied to streami
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20223#discussion_r161022921
--- Diff:
core/src/test/java/org/apache/spark/launcher/SparkLauncherSuite.java ---
@@ -137,7 +139,9 @@ public void testInProcessLauncher() throws Exception {
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20223#discussion_r161022671
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/LauncherConnection.java ---
@@ -95,15 +95,15 @@ protected synchronized void send(Message msg) thro
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20223#discussion_r161022567
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/AbstractAppHandle.java ---
@@ -91,10 +92,15 @@ LauncherConnection getConnection() {
retur
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/20227#discussion_r161022261
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/AlreadyExistException.scala
---
@@ -33,6 +33,9 @@ class TableAlreadyExis
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20222
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/20227#discussion_r161022213
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/AlreadyExistException.scala
---
@@ -33,6 +33,9 @@ class TableAlreadyExis
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20222
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85966/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20222
**[Test build #85966 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85966/testReport)**
for PR 20222 at commit
[`5eded03`](https://github.com/apache/spark/commit/5
Github user dongjoon-hyun closed the pull request at:
https://github.com/apache/spark/pull/18991
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/18991
Ur, it's not record-level filtering. Maybe, it's because I explained it too
abstractly
[here](https://github.com/apache/spark/pull/19943#discussion_r160251456
). It's stripe-level. So, the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20229
**[Test build #85978 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85978/testReport)**
for PR 20229 at commit
[`17854de`](https://github.com/apache/spark/commit/17
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20192#discussion_r161020380
--- Diff:
resource-managers/kubernetes/docker/src/main/dockerfiles/executor/Dockerfile ---
@@ -1,35 +0,0 @@
-#
-# Licensed to the Apache Software Fou
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20204
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/20204
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85969/
Test FAILed.
---
Github user attilapiros commented on the issue:
https://github.com/apache/spark/pull/20203
@tgravescs: You are welcome.
Running Spark as (with lower application.maxFailedTasksPerExecutor than
stage.maxFailedTasksPerExecutor):
```
$ bin/spark-shell --master "local-clu
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20204
**[Test build #85969 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85969/testReport)**
for PR 20204 at commit
[`e8e7112`](https://github.com/apache/spark/commit/e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20230
**[Test build #85977 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85977/testReport)**
for PR 20230 at commit
[`cc3321c`](https://github.com/apache/spark/commit/cc
Github user shaneknapp commented on the issue:
https://github.com/apache/spark/pull/20222
@gatorsmile -- the timeout is currently at *six hours*. i am loathe to
bump it any higher.
---
-
To unsubscribe, e-mail: rev
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20230
Retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/20230#discussion_r161016953
--- Diff: external/docker/spark-test/base/Dockerfile ---
@@ -15,14 +15,14 @@
# limitations under the License.
#
-FROM ubuntu:precis
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20153
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/20153
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85968/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20153
**[Test build #85968 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85968/testReport)**
for PR 20153 at commit
[`b8a700d`](https://github.com/apache/spark/commit/b
Github user smurakozi commented on a diff in the pull request:
https://github.com/apache/spark/pull/20203#discussion_r161009520
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/TaskSetBlacklistSuite.scala ---
@@ -110,7 +131,13 @@ class TaskSetBlacklistSuite extends SparkFun
Github user smurakozi commented on a diff in the pull request:
https://github.com/apache/spark/pull/20203#discussion_r161001141
--- Diff:
core/src/main/scala/org/apache/spark/status/AppStatusListener.scala ---
@@ -211,6 +211,11 @@ private[spark] class AppStatusListener(
up
Github user smurakozi commented on a diff in the pull request:
https://github.com/apache/spark/pull/20203#discussion_r161009908
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/TaskSetBlacklistSuite.scala ---
@@ -157,13 +187,19 @@ class TaskSetBlacklistSuite extends SparkFu
Github user smurakozi commented on a diff in the pull request:
https://github.com/apache/spark/pull/20203#discussion_r161005667
--- Diff:
core/src/main/scala/org/apache/spark/status/AppStatusListener.scala ---
@@ -223,6 +228,15 @@ private[spark] class AppStatusListener(
up
Github user marmbrus commented on the issue:
https://github.com/apache/spark/pull/20219
This seems reasonable to me. You can already do `sql("SELECT null")`
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.ap
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/20219
But it is possible to generate NullType data right?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additiona
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/20223#discussion_r161006918
--- Diff:
core/src/test/java/org/apache/spark/launcher/SparkLauncherSuite.java ---
@@ -137,7 +139,9 @@ public void testInProcessLauncher() throws Exce
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/20223#discussion_r160989105
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/AbstractAppHandle.java ---
@@ -91,10 +92,15 @@ LauncherConnection getConnection() {
301 - 400 of 565 matches
Mail list logo