Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/14617
Thanks @tgravescs and @squito for your comments. Based on @tgravescs 's
point, looks like making them as a table column is more valid.
So I will revert back to use column and combine
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r107050116
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/jdbc/JDBCSuite.scala
---
@@ -965,9 +965,27 @@ class JDBCSuite extends SparkFunSuite
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r107050148
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/sources/TableScanSuite.scala ---
@@ -416,4 +416,22 @@ class TableScanSuite extends DataSourceTest
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r107050121
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveDDLSuite.scala
---
@@ -1860,4 +1861,115 @@ class HiveDDLSuite
}
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r107050048
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/sources/TableScanSuite.scala ---
@@ -416,4 +416,22 @@ class TableScanSuite extends DataSourceTest
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r107049979
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -2178,4 +2178,138 @@ abstract class DDLSuite extends
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r107049953
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -296,6 +311,51 @@ class SessionCatalog(
}
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r107050001
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/jdbc/JDBCSuite.scala
---
@@ -965,9 +965,27 @@ class JDBCSuite extends SparkFunSuite
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r107049695
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -2178,4 +2178,138 @@ abstract class DDLSuite extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r107049525
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -2178,4 +2178,138 @@ abstract class DDLSuite extends
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17350#discussion_r107049220
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ComplexTypeSuite.scala
---
@@ -251,6 +251,9 @@ class ComplexTypeSuite
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17350
@hvanhovell could you check?
---
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 jerryshao commented on the issue:
https://github.com/apache/spark/pull/17335
Thanks @yaooqinn , that's really an issue here. That was my concern when I
had this fix, since we wrap the whole `SessionState.start` with real user, it
means all the operations inside this
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r107048698
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -296,6 +311,51 @@ class SessionCatalog(
}
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/17306
Yeah, that's true in this case. You could just build the grids separately
and combine them like this
```scala
val pipeline1_grid = new ParamGridBuilder()
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17246
---
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/17354
**[Test build #74915 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74915/testReport)**
for PR 17354 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17354#discussion_r107047761
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -226,6 +226,7 @@ class SessionCatalog(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17246
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 cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17354#discussion_r107047089
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -226,6 +226,7 @@ class SessionCatalog(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17246
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74913/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17246
**[Test build #74913 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74913/testReport)**
for PR 17246 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17219
**[Test build #74914 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74914/testReport)**
for PR 17219 at commit
Github user tcondie commented on a diff in the pull request:
https://github.com/apache/spark/pull/17219#discussion_r107045633
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/Trigger.scala ---
@@ -38,6 +38,51 @@ sealed trait Trigger
/**
* ::
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17246
**[Test build #74913 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74913/testReport)**
for PR 17246 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16971
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/16971
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74908/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16971
**[Test build #74908 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74908/testReport)**
for PR 16971 at commit
Github user markhamstra commented on a diff in the pull request:
https://github.com/apache/spark/pull/17297#discussion_r107044660
--- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala
---
@@ -929,12 +946,22 @@ class DAGScheduler(
}
}
Github user markhamstra commented on a diff in the pull request:
https://github.com/apache/spark/pull/17297#discussion_r107044272
--- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala
---
@@ -803,6 +810,16 @@ class DAGScheduler(
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17354
cc @cloud-fan @yhuai @windpiger
---
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 gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17354#discussion_r107043195
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -226,6 +226,7 @@ class SessionCatalog(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17354
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/17354
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74904/
Test PASSed.
---
Github user markhamstra commented on a diff in the pull request:
https://github.com/apache/spark/pull/17297#discussion_r107040190
--- Diff: core/src/main/scala/org/apache/spark/MapOutputTracker.scala ---
@@ -418,6 +424,15 @@ private[spark] class MapOutputTrackerMaster(conf:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17366
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/17366
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74907/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17366
**[Test build #74907 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74907/consoleFull)**
for PR 17366 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17295
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/17295
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74905/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17295
**[Test build #74905 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74905/testReport)**
for PR 17295 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17295
**[Test build #74911 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74911/testReport)**
for PR 17295 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/17295
retest 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 wishes so,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17295
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/17295
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74906/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17295
**[Test build #74906 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74906/testReport)**
for PR 17295 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17361
**[Test build #74910 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74910/testReport)**
for PR 17361 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17250
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/17250
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74909/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17250
**[Test build #74909 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74909/testReport)**
for PR 17250 at commit
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/17354#discussion_r107032174
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -226,6 +226,7 @@ class SessionCatalog(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17364
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/17364
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74903/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17364
**[Test build #74903 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74903/testReport)**
for PR 17364 at commit
Github user mallman commented on a diff in the pull request:
https://github.com/apache/spark/pull/16499#discussion_r107028767
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -1048,7 +1065,7 @@ private[spark] class BlockManager(
try {
Github user leifker commented on the issue:
https://github.com/apache/spark/pull/17306
Sorry of the delayed response @BryanCutler, that's pretty neat, however
this will perform unneeded work as it will execute nonsensical combinations of
parameters. For example, if pipeline2 is
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12004
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/12004
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74899/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12004
**[Test build #74899 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74899/testReport)**
for PR 12004 at commit
Github user budde commented on the issue:
https://github.com/apache/spark/pull/17250
@brkyvz Updated the PR to remove ```defaultMessageHandler()``` from
```KinesisUtils``` in favor of keeping this method in
```KinesisInputDStream```. My thought here was that this would be a better
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17250
**[Test build #74909 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74909/testReport)**
for PR 17250 at commit
Github user steveloughran commented on the issue:
https://github.com/apache/spark/pull/12004
The latest patch embraces the fact that 2.6 is the base hadoop version so
the `hadoop-aws` JAR is always pulled in, dependencies set up. One thing to
bear in mind here that the [Phase I
Github user budde commented on a diff in the pull request:
https://github.com/apache/spark/pull/17250#discussion_r107025906
--- Diff:
external/kinesis-asl/src/main/scala/org/apache/spark/streaming/kinesis/KinesisInputDStream.scala
---
@@ -71,7 +75,256 @@ private[kinesis] class
Github user steveloughran commented on the issue:
https://github.com/apache/spark/pull/14731
Any more comments?
---
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/16971
**[Test build #74908 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74908/testReport)**
for PR 16971 at commit
Github user steveloughran commented on the issue:
https://github.com/apache/spark/pull/17364
Note that as [the exception
handler](https://github.com/apache/spark/blob/master/sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileFormatWriter.scala#L244)
tries to close
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16971
retest 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 wishes
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17315
LGTM except a major comment about FAIL_FAST_MODE.
Looks great! Glad to see such a code clean!
---
If your project is set up for it, you can reply to this email and have your
reply
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17366
**[Test build #74907 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74907/consoleFull)**
for PR 17366 at commit
GitHub user dongjoon-hyun opened a pull request:
https://github.com/apache/spark/pull/17366
[SPARK-19970][SQL][BRANCH-1.6] Table owner should be USER instead of
PRINCIPAL in kerberized clusters
## What changes were proposed in this pull request?
In the kerberized hadoop
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17343
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 gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17315#discussion_r107020700
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/FailureSafeParser.scala
---
@@ -0,0 +1,80 @@
+/*
+ * Licensed to the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17343
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74897/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17343
**[Test build #74897 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74897/testReport)**
for PR 17343 at commit
Github user ioana-delaney commented on a diff in the pull request:
https://github.com/apache/spark/pull/15363#discussion_r107019056
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/StarJoinReorderSuite.scala
---
@@ -0,0 +1,580 @@
+/*
+ *
Github user markhamstra commented on a diff in the pull request:
https://github.com/apache/spark/pull/17297#discussion_r107018874
--- Diff: core/src/main/scala/org/apache/spark/MapOutputTracker.scala ---
@@ -378,15 +382,17 @@ private[spark] class MapOutputTrackerMaster(conf:
Github user ioana-delaney commented on a diff in the pull request:
https://github.com/apache/spark/pull/15363#discussion_r107018483
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/StarJoinReorderSuite.scala
---
@@ -0,0 +1,580 @@
+/*
+ *
Github user markhamstra commented on a diff in the pull request:
https://github.com/apache/spark/pull/17297#discussion_r107018555
--- Diff: core/src/main/scala/org/apache/spark/MapOutputTracker.scala ---
@@ -378,15 +382,17 @@ private[spark] class MapOutputTrackerMaster(conf:
Github user ioana-delaney commented on a diff in the pull request:
https://github.com/apache/spark/pull/15363#discussion_r107018102
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/joins.scala
---
@@ -20,19 +20,340 @@ package
Github user markhamstra commented on a diff in the pull request:
https://github.com/apache/spark/pull/17297#discussion_r107017201
--- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala
---
@@ -1265,64 +1280,11 @@ class DAGScheduler(
val
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17365
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 gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17315#discussion_r107014888
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/UnivocityParser.scala
---
@@ -233,81 +187,41 @@ class UnivocityParser(
GitHub user yupbank opened a pull request:
https://github.com/apache/spark/pull/17365
[SPARK-19962] [MLlib] [WIP] add DictVectorizer to ml.feature
## What changes were proposed in this pull request?
add a new estimator`DictVectorizer` and transformer `DictVectorizerModel`
for
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17315#discussion_r107014069
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/UnivocityParser.scala
---
@@ -46,85 +46,39 @@ class UnivocityParser(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17315#discussion_r107013706
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameReader.scala ---
@@ -435,14 +442,20 @@ class DataFrameReader private[sql](sparkSession:
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17315#discussion_r107013316
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/FailureSafeParser.scala
---
@@ -0,0 +1,80 @@
+/*
+ * Licensed to the
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17315#discussion_r107013197
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/FailureSafeParser.scala
---
@@ -0,0 +1,80 @@
+/*
+ * Licensed to the
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17170#discussion_r107013042
--- Diff: R/pkg/R/mllib_fpm.R ---
@@ -0,0 +1,153 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17170#discussion_r107012724
--- Diff: R/pkg/R/mllib_fpm.R ---
@@ -0,0 +1,144 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17343
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/17343
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74893/
Test PASSed.
---
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17170#discussion_r107010797
--- Diff: R/pkg/R/mllib_fpm.R ---
@@ -0,0 +1,153 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17295
**[Test build #74906 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74906/testReport)**
for PR 17295 at commit
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17170#discussion_r107009625
--- Diff: R/pkg/R/mllib_fpm.R ---
@@ -0,0 +1,153 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17343
**[Test build #74893 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74893/testReport)**
for PR 17343 at commit
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17170#discussion_r107009205
--- Diff: R/pkg/R/mllib_fpm.R ---
@@ -0,0 +1,153 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17170#discussion_r107009471
--- Diff: R/pkg/R/mllib_fpm.R ---
@@ -0,0 +1,153 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17170#discussion_r107011970
--- Diff: mllib/src/main/scala/org/apache/spark/ml/r/FPGrowthWrapper.scala
---
@@ -0,0 +1,86 @@
+/*
+ * Licensed to the Apache Software
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17170#discussion_r107010057
--- Diff: R/pkg/R/mllib_fpm.R ---
@@ -0,0 +1,153 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17170#discussion_r107011745
--- Diff: mllib/src/main/scala/org/apache/spark/ml/r/FPGrowthWrapper.scala
---
@@ -0,0 +1,86 @@
+/*
+ * Licensed to the Apache Software
201 - 300 of 635 matches
Mail list logo