Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19861
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84603/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19861
**[Test build #84607 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84607/testReport)**
for PR 19861 at commit
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19813#discussion_r155578442
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/ExpressionCodegen.scala
---
@@ -0,0 +1,237 @@
+/*
+ *
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/19890
We'll need to back port to 2.2/2.1 to match the previous janino update.
I'll do that. A different PR will be required for 2.1
---
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19920
Please clean up all of the template from the PR description.
ok to test
---
-
To unsubscribe, e-mail:
Github user CodingCat commented on a diff in the pull request:
https://github.com/apache/spark/pull/19864#discussion_r155587468
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/columnar/InMemoryColumnarQuerySuite.scala
---
@@ -479,4 +481,32 @@ class
Github user CodingCat commented on a diff in the pull request:
https://github.com/apache/spark/pull/19864#discussion_r155587431
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/columnar/InMemoryColumnarQuerySuite.scala
---
@@ -479,4 +481,32 @@ class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19864
**[Test build #84608 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84608/testReport)**
for PR 19864 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19783#discussion_r155561210
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -332,8 +332,41 @@ case
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19813#discussion_r155574067
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/ExpressionCodegen.scala
---
@@ -0,0 +1,237 @@
+/*
+ *
Github user foxish commented on the issue:
https://github.com/apache/spark/pull/19717
Thanks to all the reviewers, @liyinan926 and @mccheah for the comments &
fixes. This looks real close!
@ifilonenko - if we could get our integration tests to run once against
this PR this week
Github user skambha commented on the issue:
https://github.com/apache/spark/pull/19902
great! Thanks @gatorsmile , @cloud-fan.
Yes. I would be happy to open a new PR to add the rest of them.
If I compare with the getXXX types in SparkConf, it looks like the only
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19864
**[Test build #84608 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84608/testReport)**
for PR 19864 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19864
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84608/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19864
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19499
**[Test build #84604 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84604/testReport)**
for PR 19499 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19783#discussion_r155564302
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -332,8 +332,41 @@ case
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/19854
Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19717
LGTM too, thanks for the awesome work!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19783
LGTM overall
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19783#discussion_r155571394
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/FilterEstimationSuite.scala
---
@@ -359,7 +371,7 @@ class
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19920
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user smurakozi commented on the issue:
https://github.com/apache/spark/pull/19920
PR description was cleaned, thanks for catching it.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/19893#discussion_r155596370
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -34,12 +36,53 @@ abstract class SparkFunSuite
with Logging {
//
GitHub user skambha opened a pull request:
https://github.com/apache/spark/pull/19921
[SPARK-22452][SQL] Add getDouble to DataSourceV2Options
- Implemented getDouble method in DataSourceV2Options
- Add unit test
You can merge this pull request into a Git repository by running:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19921
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/19893#discussion_r155600929
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -34,12 +36,53 @@ abstract class SparkFunSuite
with Logging {
//
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19893#discussion_r155591920
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -34,12 +36,53 @@ abstract class SparkFunSuite
with Logging {
//
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19893#discussion_r155592359
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -34,12 +36,53 @@ abstract class SparkFunSuite
with Logging {
//
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19893#discussion_r155592201
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -34,12 +36,53 @@ abstract class SparkFunSuite
with Logging {
//
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19893#discussion_r155592072
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -34,12 +36,53 @@ abstract class SparkFunSuite
with Logging {
//
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19893
Also, what happens when you run tests in the sql and hive modules? Those
try to keep the same session alive for multiple suites, so I'd expect a bunch
of threads to trigger this code when they
Github user skambha commented on the issue:
https://github.com/apache/spark/pull/19902
Opened a new PR to add the getDouble method.
https://github.com/apache/spark/pull/19921
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19912
**[Test build #84605 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84605/testReport)**
for PR 19912 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19912
Thanks! Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/19893#discussion_r155601696
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -34,12 +36,53 @@ abstract class SparkFunSuite
with Logging {
//
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19893#discussion_r155602584
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -34,12 +36,53 @@ abstract class SparkFunSuite
with Logging {
//
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/19893#discussion_r155598719
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -34,12 +36,53 @@ abstract class SparkFunSuite
with Logging {
//
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19893#discussion_r155599671
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -34,12 +36,53 @@ abstract class SparkFunSuite
with Logging {
//
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19912
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84605/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19912
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19912
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/19893#discussion_r155603848
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -34,12 +36,53 @@ abstract class SparkFunSuite
with Logging {
//
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19893#discussion_r155606762
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -34,12 +36,53 @@ abstract class SparkFunSuite
with Logging {
//
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19811
**[Test build #84611 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84611/testReport)**
for PR 19811 at commit
Github user smurakozi commented on a diff in the pull request:
https://github.com/apache/spark/pull/19920#discussion_r155609528
--- Diff:
core/src/main/scala/org/apache/spark/status/AppStatusListener.scala ---
@@ -29,6 +29,7 @@ import org.apache.spark.executor.TaskMetrics
Github user henryr commented on the issue:
https://github.com/apache/spark/pull/19683
My guess is that it's safe to do so in our case because of the immediate
projection that happens. In general, emitting JoinedRows where the RHS row is
shared between all JoinedRows could be a
Github user danielvdende commented on a diff in the pull request:
https://github.com/apache/spark/pull/19911#discussion_r155622687
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/jdbc/DB2Dialect.scala ---
@@ -48,5 +48,7 @@ private object DB2Dialect extends JdbcDialect {
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/19893#discussion_r155605695
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -34,12 +36,53 @@ abstract class SparkFunSuite
with Logging {
//
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/19893#discussion_r155608281
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -34,12 +36,53 @@ abstract class SparkFunSuite
with Logging {
//
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19861
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19861
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84607/
Test PASSed.
---
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/19911
@dongjoon-hyun ok made the changes, also replaced the test that was in
place for `isCascadingTruncateTable` with one for the `getTruncateQuery`
method. Right now, I've left the method in
Github user liyinan926 commented on the issue:
https://github.com/apache/spark/pull/19717
With some bug fixes in
https://github.com/apache/spark/pull/19717/commits/83d0b9cc29eec2654308344ac8b2208e5e3f08df,
I was able to run `SparkPi` on a GKE cluster.
---
Github user gaborgsomogyi commented on the issue:
https://github.com/apache/spark/pull/19893
Yeah, this is fully true. This enhancement is definitely will not solve the
issues once and for all. The problems were hidden till now and we would like to
make a step ahead and make it at
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19920#discussion_r155603153
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/HistoryServer.scala ---
@@ -188,11 +188,11 @@ class HistoryServer(
}
def
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19920#discussion_r155603588
--- Diff: core/src/main/scala/org/apache/spark/status/AppStatusStore.scala
---
@@ -25,6 +25,7 @@ import scala.collection.JavaConverters._
import
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19920#discussion_r155604279
--- Diff: core/src/main/scala/org/apache/spark/status/api/v1/api.scala ---
@@ -24,27 +24,32 @@ import
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19920#discussion_r155603886
--- Diff: core/src/main/scala/org/apache/spark/status/api/v1/api.scala ---
@@ -24,27 +24,32 @@ import
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19920#discussion_r155603905
--- Diff: core/src/main/scala/org/apache/spark/status/api/v1/api.scala ---
@@ -24,27 +24,32 @@ import
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19920#discussion_r155603357
--- Diff:
core/src/main/scala/org/apache/spark/status/AppStatusListener.scala ---
@@ -29,6 +29,7 @@ import org.apache.spark.executor.TaskMetrics
import
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19893
> All in all if you have an approach which solves context reuse feel free
to share.
I don't have one, but it feels sub-optimal to add code that will knowingly
trigger false positives in a
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19893#discussion_r155609975
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -34,12 +36,53 @@ abstract class SparkFunSuite
with Logging {
//
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19920#discussion_r155610181
--- Diff:
core/src/main/scala/org/apache/spark/status/AppStatusListener.scala ---
@@ -29,6 +29,7 @@ import org.apache.spark.executor.TaskMetrics
import
Github user smurakozi commented on a diff in the pull request:
https://github.com/apache/spark/pull/19920#discussion_r155610889
--- Diff: core/src/main/scala/org/apache/spark/status/api/v1/api.scala ---
@@ -24,27 +24,32 @@ import
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19920#discussion_r155611800
--- Diff: core/src/main/scala/org/apache/spark/status/api/v1/api.scala ---
@@ -24,27 +24,32 @@ import
Github user smurakozi commented on a diff in the pull request:
https://github.com/apache/spark/pull/19920#discussion_r155611738
--- Diff: core/src/main/scala/org/apache/spark/status/api/v1/api.scala ---
@@ -24,27 +24,32 @@ import
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19864
**[Test build #84612 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84612/testReport)**
for PR 19864 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19861
**[Test build #84607 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84607/testReport)**
for PR 19861 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19717
**[Test build #84613 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84613/testReport)**
for PR 19717 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19683
**[Test build #84614 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84614/testReport)**
for PR 19683 at commit
Github user bolkedebruin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19911#discussion_r155619800
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/jdbc/DB2Dialect.scala ---
@@ -48,5 +48,7 @@ private object DB2Dialect extends JdbcDialect {
Github user bolkedebruin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19911#discussion_r155619721
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/jdbc/DB2Dialect.scala ---
@@ -48,5 +48,7 @@ private object DB2Dialect extends JdbcDialect {
Github user uzadude commented on the issue:
https://github.com/apache/spark/pull/19683
@henryr I understand what you're saying. I'm not sure why there is the
UnsafeProject in the end of the function, but it's commented in this PR that
fixes [SPARK-13476] without much elaboration.
Github user bolkedebruin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19911#discussion_r155624133
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/jdbc/DB2Dialect.scala ---
@@ -48,5 +48,7 @@ private object DB2Dialect extends JdbcDialect {
GitHub user srowen opened a pull request:
https://github.com/apache/spark/pull/19922
[SPARK-22688][SQL][HOTFIX] Upgrade Janino version to 3.0.8
## What changes were proposed in this pull request?
Hotfix inadvertent change to xmlbuilder dep when updating Janino.
See
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19884
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84616/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19884
**[Test build #84616 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84616/testReport)**
for PR 19884 at commit
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/19884#discussion_r155626249
--- Diff: python/pyspark/sql/types.py ---
@@ -1658,13 +1657,13 @@ def from_arrow_type(at):
spark_type = FloatType()
elif at ==
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/19893#discussion_r155626333
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -34,12 +36,53 @@ abstract class SparkFunSuite
with Logging {
//
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19884
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19811
**[Test build #84610 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84610/testReport)**
for PR 19811 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19811
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84610/
Test FAILed.
---
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/19843#discussion_r155629359
--- Diff: mllib/src/test/scala/org/apache/spark/ml/util/MLTest.scala ---
@@ -0,0 +1,92 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/19843#discussion_r155629428
--- Diff: mllib/src/test/scala/org/apache/spark/ml/util/MLTest.scala ---
@@ -0,0 +1,92 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18995
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user smurakozi commented on a diff in the pull request:
https://github.com/apache/spark/pull/19920#discussion_r155631788
--- Diff: core/src/main/scala/org/apache/spark/status/AppStatusStore.scala
---
@@ -25,6 +25,7 @@ import scala.collection.JavaConverters._
import
Github user smurakozi commented on a diff in the pull request:
https://github.com/apache/spark/pull/19920#discussion_r155631687
--- Diff:
core/src/main/scala/org/apache/spark/status/AppStatusListener.scala ---
@@ -29,6 +29,7 @@ import org.apache.spark.executor.TaskMetrics
Github user smurakozi commented on the issue:
https://github.com/apache/spark/pull/19906
Thanks for your help and reviews @HyukjinKwon, @vanzin
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19843#discussion_r155636618
--- Diff: mllib/src/test/scala/org/apache/spark/ml/util/MLTest.scala ---
@@ -0,0 +1,81 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19884
**[Test build #84616 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84616/testReport)**
for PR 19884 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19922
**[Test build #84615 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84615/testReport)**
for PR 19922 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19811
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18995
**[Test build #84609 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84609/testReport)**
for PR 18995 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18995
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84609/
Test PASSed.
---
Github user smurakozi commented on a diff in the pull request:
https://github.com/apache/spark/pull/19920#discussion_r155631377
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/HistoryServer.scala ---
@@ -188,11 +188,11 @@ class HistoryServer(
}
Github user smurakozi commented on a diff in the pull request:
https://github.com/apache/spark/pull/19920#discussion_r155632345
--- Diff: core/src/main/scala/org/apache/spark/status/api/v1/api.scala ---
@@ -24,27 +24,32 @@ import
401 - 497 of 497 matches
Mail list logo
