Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16781
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16781
**[Test build #76141 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76141/testReport)**
for PR 16781 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17724
**[Test build #76146 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76146/testReport)**
for PR 17724 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17761
**[Test build #76145 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76145/testReport)**
for PR 17761 at commit
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17728#discussion_r113278764
--- Diff: R/pkg/R/DataFrame.R ---
@@ -3642,3 +3643,74 @@ setMethod("checkpoint",
df <- callJMethod(x@sdf, "checkpoint",
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17728#discussion_r113278677
--- Diff: R/pkg/R/DataFrame.R ---
@@ -3642,3 +3643,74 @@ setMethod("checkpoint",
df <- callJMethod(x@sdf, "checkpoint",
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17728#discussion_r113278517
--- Diff: R/pkg/R/DataFrame.R ---
@@ -3642,3 +3643,74 @@ setMethod("checkpoint",
df <- callJMethod(x@sdf, "checkpoint",
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17596
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76136/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17596
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17596
**[Test build #76136 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76136/testReport)**
for PR 17596 at commit
Github user kunalkhamar commented on a diff in the pull request:
https://github.com/apache/spark/pull/17761#discussion_r113274024
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/CachedKafkaConsumer.scala
---
@@ -62,11 +63,20 @@ private[kafka010]
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17761
**[Test build #76144 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76144/testReport)**
for PR 17761 at commit
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/17761#discussion_r113273935
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/CachedKafkaConsumer.scala
---
@@ -62,11 +63,20 @@ private[kafka010] case class
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/17761#discussion_r113273103
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -84,7 +86,20 @@ private[spark] class Executor(
}
// Start
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15125
---
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 zsxwing opened a pull request:
https://github.com/apache/spark/pull/17761
[SPARK-20461][Core][SS]Use UninterruptibleThread for Executor and fix the
potential hang in CachedKafkaConsumer
## What changes were proposed in this pull request?
This PR changes
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/15125
merged to master/2.2
I talked to Ankur about this change offline.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17728#discussion_r113270143
--- Diff: R/pkg/R/DataFrame.R ---
@@ -1321,7 +1321,7 @@ setMethod("toRDD",
#' Groups the SparkDataFrame using the specified columns, so we can run
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17757#discussion_r113269591
--- Diff: R/pkg/inst/tests/testthat/test_mllib_classification.R ---
@@ -284,22 +284,11 @@ test_that("spark.mlp", {
c("1.0", "1.0",
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17725
**[Test build #76143 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76143/testReport)**
for PR 17725 at commit
Github user dbtsai commented on the issue:
https://github.com/apache/spark/pull/17757
LGTM. Please merge the current master to resolve the conflicts.
---
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 dbtsai commented on the issue:
https://github.com/apache/spark/pull/17746
@srowen Couple APIs changes in Breeze 0.13 are not source code compatible
with 0.12. We should tell users about that in the release note, and they need
to do their migration if they are using Breeze
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/17746
Not that I have any specific concern, but did anyone look at the changes
from 0.12 to 0.13 to see if anything might be breaking? probably not, but it
does leak into the user classpath
---
If your
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/17759
Yes, but we generally ask people to do a more comprehensive sweep for typos
if they're going to bother. It's a lot of overhead to merge a bunch of these
---
If your project is set up for it, you
Github user dbtsai commented on the issue:
https://github.com/apache/spark/pull/17746
Many thanks for @WeichenXu123 helping to fix this bug in breeze!
---
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 dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/17746#discussion_r113256277
--- Diff: R/pkg/inst/tests/testthat/test_mllib_classification.R ---
@@ -288,18 +288,18 @@ test_that("spark.mlp", {
c(0, 0, 0, 0, 0, 5, 5, 5, 5, 5,
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17746
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17758
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76138/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17758
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17758
**[Test build #76138 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76138/testReport)**
for PR 17758 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17596
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76134/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17596
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17596
**[Test build #76134 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76134/testReport)**
for PR 17596 at commit
Github user dbtsai commented on the issue:
https://github.com/apache/spark/pull/17746
LGTM. Merged into master and branch 2.2
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17649
**[Test build #76142 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76142/testReport)**
for PR 17649 at commit
Github user squito commented on the issue:
https://github.com/apache/spark/pull/16781
@ueshin updated per your feedback.
I should have explained that the last update *did* handle partition tables
(it added the second call to `getStorageTzOptions` in `HiveMetastoreCatalog`),
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16781
**[Test build #76141 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76141/testReport)**
for PR 16781 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17760
**[Test build #76140 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76140/testReport)**
for PR 17760 at commit
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/17760
[SPARK-20439] [SQL] [Backport-2.1] Fix Catalog API listTables and getTable
when failed to fetch table metadata
### What changes were proposed in this pull request?
This PR is to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17649
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17649
**[Test build #76139 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76139/testReport)**
for PR 17649 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17649
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76139/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17649
**[Test build #76139 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76139/testReport)**
for PR 17649 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17759
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 mineo opened a pull request:
https://github.com/apache/spark/pull/17759
[DOCS] Fix a typo in Encoder.clsTag
## What changes were proposed in this pull request?
Fixes a typo: `and` -> `an`
## How was this patch tested?
Not at all.
You can
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17758
**[Test build #76138 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76138/testReport)**
for PR 17758 at commit
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/17727
Hm I don't think the comment makes sense ...
---
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/17758
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/17758
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76137/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17758
**[Test build #76137 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76137/testReport)**
for PR 17758 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17758
**[Test build #76137 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76137/testReport)**
for PR 17758 at commit
GitHub user maropu opened a pull request:
https://github.com/apache/spark/pull/17758
[SPARK-20460][SQL] Make it more consistent to handle column name duplication
## What changes were proposed in this pull request?
This pr made it more consistent to handle column name
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/17540
@zsxwing, what do you think the right way forward is? The "problem" with
this patch seems to actually be a wide-spread problem with the data sources API
-- it doesn't correctly link into SQL plans.
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/17540#discussion_r113225697
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileFormatWriter.scala
---
@@ -161,50 +161,51 @@ object FileFormatWriter
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17728
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/17728
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76135/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17728
**[Test build #76135 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76135/testReport)**
for PR 17728 at commit
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/17130#discussion_r113220960
--- Diff: mllib/src/main/scala/org/apache/spark/ml/fpm/FPGrowth.scala ---
@@ -268,12 +269,8 @@ class FPGrowthModel private[ml] (
val predictUDF =
Github user Krimit commented on the issue:
https://github.com/apache/spark/pull/17673
I'm happy to take a look! I'll have some time to dig in deeper tomorrow.
Some of my initial impressions:
* There's a lot going on here, I agree with @hhbyyh that it would be
cleaner to put the
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/17757#discussion_r113215088
--- Diff: R/pkg/inst/tests/testthat/test_mllib_classification.R ---
@@ -284,22 +284,11 @@ test_that("spark.mlp", {
c("1.0", "1.0",
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17596
**[Test build #76136 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76136/testReport)**
for PR 17596 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17728
**[Test build #76135 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76135/testReport)**
for PR 17728 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17596
**[Test build #76134 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76134/testReport)**
for PR 17596 at commit
Github user lw-lin commented on the issue:
https://github.com/apache/spark/pull/17735
@zsxwing @brkyvz would you take a look at this? thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/17757
@HyukjinKwon We hit this issue at #17746 when we upgrade breeze to 0.13.1.
Since these tests don't converged, we check intermediate result which is
vulnerable, so I switched to check the last
Github user zero323 commented on the issue:
https://github.com/apache/spark/pull/17728
I actually. I wasn't sure about this in the first place. Let me spread it
between `details` and `examples`.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17757
Hi @yanboliang, do you mind if I ask why they were flaky? I am just curious
and want to know.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/17658#discussion_r113196872
--- Diff: core/src/main/scala/org/apache/spark/ui/SparkUI.scala ---
@@ -139,6 +140,8 @@ private[spark] abstract class SparkUITab(parent:
SparkUI, prefix:
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/17658
Jenkins, test 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 tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/17748#discussion_r113192444
--- Diff:
common/network-yarn/src/main/java/org/apache/spark/network/yarn/YarnShuffleService.java
---
@@ -363,25 +362,29 @@ protected File
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17757
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76133/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17757
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17757
**[Test build #76133 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76133/testReport)**
for PR 17757 at commit
Github user n-marion commented on a diff in the pull request:
https://github.com/apache/spark/pull/17686#discussion_r113188759
--- Diff: core/src/main/scala/org/apache/spark/ui/UIUtils.scala ---
@@ -527,4 +530,24 @@ private[spark] object UIUtils extends Logging {
Github user n-marion commented on a diff in the pull request:
https://github.com/apache/spark/pull/17686#discussion_r113188711
--- Diff: core/src/main/scala/org/apache/spark/ui/UIUtils.scala ---
@@ -27,13 +27,16 @@ import scala.xml.transform.{RewriteRule,
RuleTransformer}
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17757
**[Test build #76133 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76133/testReport)**
for PR 17757 at commit
GitHub user yanboliang opened a pull request:
https://github.com/apache/spark/pull/17757
[Minor][ML] Fix some PySpark & SparkR flaky tests
## What changes were proposed in this pull request?
Fix some PySpark & SparkR flaky tests.
## How was this patch tested?
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/17727
@rxin would be able to judge 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 srowen commented on the issue:
https://github.com/apache/spark/pull/17542
Ping @w3iBStime can you update this or close?
---
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 n-marion commented on a diff in the pull request:
https://github.com/apache/spark/pull/17686#discussion_r113173559
--- Diff: core/src/main/scala/org/apache/spark/ui/UIUtils.scala ---
@@ -527,4 +530,24 @@ private[spark] object UIUtils extends Logging {
Github user n-marion commented on a diff in the pull request:
https://github.com/apache/spark/pull/17686#discussion_r113172804
--- Diff: core/src/main/scala/org/apache/spark/ui/UIUtils.scala ---
@@ -27,13 +27,16 @@ import scala.xml.transform.{RewriteRule,
RuleTransformer}
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17744
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76132/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17744
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17744
**[Test build #76132 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76132/testReport)**
for PR 17744 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17744
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76131/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17744
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17744
**[Test build #76131 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76131/testReport)**
for PR 17744 at commit
Github user xuanyuanking commented on the issue:
https://github.com/apache/spark/pull/17702
cc @zsxwing @tdas, can you review this? Founded the relative code of yours
before. Thanks :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17737
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/17737
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76130/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17737
**[Test build #76130 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76130/testReport)**
for PR 17737 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17744
**[Test build #76132 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76132/testReport)**
for PR 17744 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17754
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17744
**[Test build #76131 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76131/testReport)**
for PR 17744 at commit
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/17754
Merged into master and branch-2.2. Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17686#discussion_r113133777
--- Diff: core/src/main/scala/org/apache/spark/ui/UIUtils.scala ---
@@ -527,4 +530,24 @@ private[spark] object UIUtils extends Logging {
origHref
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17686#discussion_r113133450
--- Diff: core/src/main/scala/org/apache/spark/ui/UIUtils.scala ---
@@ -27,13 +27,16 @@ import scala.xml.transform.{RewriteRule,
RuleTransformer}
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17686#discussion_r113133641
--- Diff: core/src/main/scala/org/apache/spark/ui/UIUtils.scala ---
@@ -527,4 +530,24 @@ private[spark] object UIUtils extends Logging {
origHref
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/17746#discussion_r113133446
--- Diff: R/pkg/inst/tests/testthat/test_mllib_classification.R ---
@@ -288,18 +288,18 @@ test_that("spark.mlp", {
c(0, 0, 0, 0, 0, 5, 5, 5, 5,
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17740
---
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
201 - 300 of 325 matches
Mail list logo