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 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 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/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 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/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 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 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 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 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 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 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/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 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 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 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 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 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
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 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 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 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 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 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 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 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 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 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 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 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/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 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 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
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 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/20229
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/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 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/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 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 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 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 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/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
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 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 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 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 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 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 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 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 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 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 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
@BryanCutler I think this comes up in the Arrow sync yesterday
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.
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 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 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 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 vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20167#discussion_r161075359
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -80,10 +80,27 @@ trait MesosSched
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20167#discussion_r161075101
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -80,10 +80,27 @@ trait MesosSched
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20167#discussion_r161074854
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -80,10 +80,27 @@ trait MesosSched
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20167#discussion_r161075477
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -80,10 +80,27 @@ trait MesosSched
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20208
**[Test build #85983 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85983/testReport)**
for PR 20208 at commit
[`499801e`](https://github.com/apache/spark/commit/4
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/20225#discussion_r161077122
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/continuous/ContinuousSuite.scala
---
@@ -219,6 +201,44 @@ class ContinuousSuite extends Conti
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20208
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/20208
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85983/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20203
**[Test build #85991 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85991/testReport)**
for PR 20203 at commit
[`0120955`](https://github.com/apache/spark/commit/01
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20225
**[Test build #85990 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85990/testReport)**
for PR 20225 at commit
[`ad2f206`](https://github.com/apache/spark/commit/ad
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19054
**[Test build #85985 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85985/testReport)**
for PR 19054 at commit
[`c689ff1`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19054
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/19054
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85985/
Test PASSed.
---
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/20146
@viirya
Discuss with @jkbradley offline, we're now busy fixing some issues (e.g.
#20238) in ML structured streaming support, it looks bad after the code freeze,
and we may not be able to m
Github user sameeragarwal commented on the issue:
https://github.com/apache/spark/pull/20231
test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20231
**[Test build #85992 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85992/testReport)**
for PR 20231 at commit
[`39df77f`](https://github.com/apache/spark/commit/39
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/20238
Reviewing now
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@
Github user icexelloss commented on a diff in the pull request:
https://github.com/apache/spark/pull/20211#discussion_r161085442
--- Diff: python/pyspark/sql/group.py ---
@@ -233,6 +233,27 @@ def apply(self, udf):
| 2| 1.1094003924504583|
+---+
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20222
**[Test build #85980 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85980/testReport)**
for PR 20222 at commit
[`5eded03`](https://github.com/apache/spark/commit/5
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 icexelloss commented on a diff in the pull request:
https://github.com/apache/spark/pull/20211#discussion_r161086349
--- Diff: python/pyspark/sql/group.py ---
@@ -233,6 +233,27 @@ def apply(self, udf):
| 2| 1.1094003924504583|
+---+
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/85980/
Test FAILed.
---
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/20238
LGTM
I'm going to merge this with master and backport it to branch-2.3 since
this fixes 1 of 2 bugs in RFormulaModel for use with Structured Streaming.
Thanks @MrBago !
---
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/19001
Now that https://github.com/apache/spark/pull/19080 has been merged to
trunk, I am rebasing this PR. A small part of this PR is put in
https://github.com/apache/spark/pull/20206 and ready for re
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20238
---
-
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/20223
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85984/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20223
**[Test build #85984 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85984/testReport)**
for PR 20223 at commit
[`9c7d2d3`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20225
**[Test build #85993 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85993/testReport)**
for PR 20225 at commit
[`54d3a2c`](https://github.com/apache/spark/commit/54
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20223
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/20189
@mgaido91 Ah, sorry. There is a race condition in this test since the query
may make progress when we are checking the results. Please revert to your
previous test. Thanks!
---
--
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/20138#discussion_r161082423
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -405,49 +404,70 @@ private[history] class FsHistoryProvider(conf
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20204
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20204
**[Test build #85994 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85994/testReport)**
for PR 20204 at commit
[`e8e7112`](https://github.com/apache/spark/commit/e8
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/20138#discussion_r161095561
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -405,49 +404,70 @@ private[history] class FsHistoryProvider(conf
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/20138
I haven't had a chance to read though your code, but as @squito said, I am
against any default feature that deletes files from the eventLog dir. Many
users, such as myself, use one log dir for bot
201 - 300 of 565 matches
Mail list logo
