Github user burness commented on the issue:
https://github.com/apache/spark/pull/13116
@HyukjinKwon It is too hard to solve the OOM, I'm so sorry
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user budde commented on the issue:
https://github.com/apache/spark/pull/17250
@brkyvz I think if we're eliminating the constructor arguments then the
second approach you've proposed might make more sense. I can't think of
anything cleaner.
---
If your project is set up for
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/16954#discussion_r105831346
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala
---
@@ -123,19 +123,36 @@ case class Not(child:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17109
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74488/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17109
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/17109
**[Test build #74488 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74488/testReport)**
for PR 17109 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17277
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/17277
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74482/
Test PASSed.
---
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16373#discussion_r105831078
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -925,6 +925,26 @@ class DDLSuite extends QueryTest with
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17277
**[Test build #74482 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74482/testReport)**
for PR 17277 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16373#discussion_r105830656
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -925,6 +925,26 @@ class DDLSuite extends QueryTest with
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17277
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74480/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17277
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/17277
**[Test build #74480 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74480/testReport)**
for PR 17277 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16954
**[Test build #74489 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74489/testReport)**
for PR 16954 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17109
**[Test build #74488 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74488/testReport)**
for PR 17109 at commit
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/16954#discussion_r105830367
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -365,17 +368,73 @@ object TypeCoercion {
Github user hhbyyh commented on the issue:
https://github.com/apache/spark/pull/13656
Close this and add the support to ml.fpm.
https://github.com/apache/spark/pull/17280
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user hhbyyh closed the pull request at:
https://github.com/apache/spark/pull/13656
---
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 cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17255#discussion_r105830235
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JsonInferSchema.scala
---
@@ -40,18 +40,11 @@ private[sql] object
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/16867
@squito
Thanks a lot for comments. I've refined :):)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16373
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74481/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16373
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/16373
**[Test build #74481 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74481/testReport)**
for PR 16373 at commit
Github user RishikeshTeke closed the pull request at:
https://github.com/apache/spark/pull/17279
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15628
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74479/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15628
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
enabled
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15628
**[Test build #74479 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74479/consoleFull)**
for PR 15628 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17267
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74487/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17267
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/17267
**[Test build #74487 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74487/testReport)**
for PR 17267 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/13116
Hi @burness, what's the state of this PR?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user uncleGen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17267#discussion_r105827541
--- Diff: python/pyspark/sql/utils.py ---
@@ -24,7 +24,7 @@ def __init__(self, desc, stackTrace):
self.stackTrace = stackTrace
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17267
**[Test build #74487 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74487/testReport)**
for PR 17267 at commit
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16373#discussion_r105827253
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -925,6 +925,26 @@ class DDLSuite extends QueryTest with
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16373#discussion_r105826784
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -925,6 +925,26 @@ class DDLSuite extends QueryTest with
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15628
**[Test build #74486 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74486/testReport)**
for PR 15628 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17175#discussion_r105825600
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryTableScanExec.scala
---
@@ -41,11 +41,31 @@ case class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17175#discussion_r105825397
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryTableScanExec.scala
---
@@ -41,11 +41,31 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17285
**[Test build #74485 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74485/testReport)**
for PR 17285 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17265
---
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 cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17265
LGTM, merging to master!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17285
**[Test build #74484 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74484/testReport)**
for PR 17285 at commit
GitHub user rxin opened a pull request:
https://github.com/apache/spark/pull/17285
[SPARK-19944][SQL] Move SQLConf from sql/core to sql/catalyst
## What changes were proposed in this pull request?
This patch moves SQLConf from sql/core to sql/catalyst. To minimize the
changes,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17241
**[Test build #74483 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74483/testReport)**
for PR 17241 at commit
Github user ouyangxiaochen commented on the issue:
https://github.com/apache/spark/pull/17270
@gatorsmile cc ,is it reasonable? 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 marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/17179#discussion_r105822080
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/KeyedState.scala ---
@@ -61,25 +65,50 @@ import
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/17179#discussion_r105821698
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/KeyValueGroupedDataset.scala ---
@@ -298,12 +368,14 @@ class KeyValueGroupedDataset[K, V]
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/17179#discussion_r105823059
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/FlatMapGroupsWithStateExec.scala
---
@@ -0,0 +1,270 @@
+/*
+ *
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/17179#discussion_r105822317
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/KeyedState.scala ---
@@ -61,25 +65,50 @@ import
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/17179#discussion_r105822109
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/KeyedState.scala ---
@@ -61,25 +65,50 @@ import
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/17179#discussion_r105821496
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/KeyValueGroupedDataset.scala ---
@@ -249,6 +250,43 @@ class KeyValueGroupedDataset[K, V]
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17240#discussion_r105823209
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -122,46 +119,48 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16867
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74476/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16867
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/16867
**[Test build #74476 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74476/testReport)**
for PR 16867 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17240#discussion_r105822819
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -122,46 +119,48 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16867
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74475/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16867
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/16867
**[Test build #74475 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74475/testReport)**
for PR 16867 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17240#discussion_r105822517
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -122,46 +119,48 @@ case class
Github user crackcell commented on the issue:
https://github.com/apache/spark/pull/17233
@jkbradley Hi, I have made some updates according to your comments, please
review it again. :-)
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17240#discussion_r105821744
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -87,8 +84,8 @@ case class
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17278#discussion_r105821394
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -140,7 +140,8 @@ abstract class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17277
**[Test build #74482 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74482/testReport)**
for PR 17277 at commit
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16373#discussion_r105820580
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -642,18 +644,34 @@ case class ShowTablesCommand(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16373
**[Test build #74481 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74481/testReport)**
for PR 16373 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17277
**[Test build #74480 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74480/testReport)**
for PR 17277 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17194
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74478/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17194
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/17194
**[Test build #74478 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74478/testReport)**
for PR 17194 at commit
Github user crackcell commented on a diff in the pull request:
https://github.com/apache/spark/pull/17233#discussion_r105820314
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/StringIndexer.scala ---
@@ -188,35 +189,45 @@ class StringIndexerModel (
Github user crackcell commented on a diff in the pull request:
https://github.com/apache/spark/pull/17233#discussion_r105820279
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/feature/StringIndexerSuite.scala ---
@@ -122,6 +122,86 @@ class StringIndexerSuite
Github user crackcell commented on a diff in the pull request:
https://github.com/apache/spark/pull/17233#discussion_r105820283
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/feature/StringIndexerSuite.scala ---
@@ -122,6 +122,86 @@ class StringIndexerSuite
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16373#discussion_r105819853
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -925,6 +925,26 @@ class DDLSuite extends QueryTest with
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15628
**[Test build #74479 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74479/consoleFull)**
for PR 15628 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/17186
@sameeragarwal Thanks for the comment. I've updated
`InferFiltersFromConstraints`.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/17274#discussion_r105818725
--- Diff: R/pkg/inst/tests/testthat/test_context.R ---
@@ -177,6 +177,13 @@ test_that("add and get file to be downloaded with
Spark job on every node",
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17251#discussion_r105818676
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -590,6 +591,22 @@ object TypeCoercion {
}
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17186#discussion_r105818486
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -190,6 +190,15 @@ object SQLConf {
.booleanConf
Github user brkyvz commented on the issue:
https://github.com/apache/spark/pull/17250
Good point @budde. I can think of two options:
1. Leave it as a constructor param
2. Make the `Builder` class non-generic and have the `build` function take
the message handler:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17186
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/17186
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74473/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17186
**[Test build #74473 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74473/testReport)**
for PR 17186 at commit
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/17240
@nsyca Thanks. I know there could be such cases when size is also useful.
However, usually big tables (fact table) have more columns than small tables,
so cardinality and size is positively
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17186#discussion_r105815354
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -190,6 +190,15 @@ object SQLConf {
.booleanConf
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17240#discussion_r105815087
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -204,63 +206,37 @@ object JoinReorderDP
Github user facaiy commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r105814881
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impurity/ApproxBernoulliImpurity.scala
---
@@ -0,0 +1,155 @@
+/*
+ * Licensed to the Apache
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17272
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74472/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17272
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/17272
**[Test build #74472 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74472/testReport)**
for PR 17272 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17277
Found a JIRA https://issues.apache.org/jira/browse/IMPALA-252 to explain
how IMPALA handles it.
**Static partition keys may not be NULL or the empty string**
So `INSERT INTO TABLE
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/17283#discussion_r105813634
--- Diff: mllib/src/test/scala/org/apache/spark/ml/fpm/FPGrowthSuite.scala
---
@@ -103,6 +103,22 @@ class FPGrowthSuite extends SparkFunSuite with
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/17283#discussion_r105813424
--- Diff: mllib/src/test/scala/org/apache/spark/ml/fpm/FPGrowthSuite.scala
---
@@ -103,6 +103,22 @@ class FPGrowthSuite extends SparkFunSuite with
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/17283#discussion_r105813550
--- Diff: mllib/src/test/scala/org/apache/spark/ml/fpm/FPGrowthSuite.scala
---
@@ -103,6 +103,22 @@ class FPGrowthSuite extends SparkFunSuite with
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17194
**[Test build #74478 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74478/testReport)**
for PR 17194 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15628
**[Test build #74477 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74477/testReport)**
for PR 15628 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15628
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74477/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15628
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 sethah commented on the issue:
https://github.com/apache/spark/pull/15628
@dbtsai Let me know your thoughts on the comments I left. Thanks for the
review!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
1 - 100 of 552 matches
Mail list logo