Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/8451#issuecomment-134868225
`Lists.newArrayList` -> `Arrays.asList` look good to me. But I don't see
why we need to replace `Strings` by `StringUtils` from commons-lang. Are we
going to remove Guava
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8450#issuecomment-134867591
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8450#issuecomment-134867593
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/7629#issuecomment-134867614
Marcelo you will need to change the timeout in the code itself for it to
increase from 175
On Aug 25, 2015 11:24 PM, "UCB AMPLab" wrote:
> Test FAILed.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8450#issuecomment-134867488
[Test build #41590 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41590/console)
for PR 8450 at commit
[`fe4c0fd`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5754#issuecomment-134866517
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 pr
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8394
---
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 enab
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8449
---
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 enab
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/7959#issuecomment-134863846
@JoshRosen Hi, I think this patch stays here too long. Is it ok to merge it?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user dbtsai commented on the pull request:
https://github.com/apache/spark/pull/8449#issuecomment-134863383
Merged into master and branch 1.5. 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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8128#issuecomment-134860816
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8128#issuecomment-134860821
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8451#issuecomment-134860535
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8128#issuecomment-134860489
[Test build #41596 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41596/console)
for PR 8128 at commit
[`50b33d8`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8451#issuecomment-134860189
[Test build #41603 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41603/console)
for PR 8451 at commit
[`71757ca`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8451#issuecomment-134860529
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
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/8390#issuecomment-134859634
@navis Nvm, thanks for working on this all the same!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8449#issuecomment-134855722
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8449#issuecomment-134855719
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8449#issuecomment-134855645
[Test build #41602 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41602/console)
for PR 8449 at commit
[`0e8d658`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7629#issuecomment-134854998
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7629#issuecomment-134855001
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7629#issuecomment-134854945
**[Test build #41584 timed
out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41584/console)**
for PR 7629 at commit
[`a2c7c59`](https://github
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/8394#issuecomment-134852794
Thanks @felixcheung -- Merging 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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8453#issuecomment-134852237
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8453#issuecomment-134852232
[Test build #41605 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41605/console)
for PR 8453 at commit
[`e8d88e9`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8453#issuecomment-134852236
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8445#issuecomment-134850445
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8445#issuecomment-134850439
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8445#issuecomment-134850117
[Test build #41600 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41600/console)
for PR 8445 at commit
[`24f9282`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8453#issuecomment-134849228
[Test build #41605 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41605/consoleFull)
for PR 8453 at commit
[`e8d88e9`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8436#issuecomment-134848565
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8436#issuecomment-134848568
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8436#issuecomment-134847882
[Test build #41598 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41598/console)
for PR 8436 at commit
[`5169ce0`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8453#issuecomment-134847440
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8453#issuecomment-134847441
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8453#issuecomment-134847430
[Test build #41604 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41604/console)
for PR 8453 at commit
[`17b7406`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8453#issuecomment-134846554
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8453#issuecomment-134846595
Merged build started.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8451#issuecomment-134845930
[Test build #41603 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41603/consoleFull)
for PR 8451 at commit
[`71757ca`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8453#issuecomment-134845813
[Test build #41604 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41604/consoleFull)
for PR 8453 at commit
[`17b7406`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8453#issuecomment-134845690
Merged build started.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8453#issuecomment-134845675
Merged build triggered.
---
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 h
GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/8453
[SPARK-10104][SQL] Consolidate different forms of table identifiers
Right now, we have QualifiedTableName, TableIdentifier, and Seq[String] to
represent table identifiers. We should only have one
Github user dbtsai commented on the pull request:
https://github.com/apache/spark/pull/8449#issuecomment-134844109
LGTM. Waiting for test result.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8451#issuecomment-134843830
Merged build started.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8451#issuecomment-134843757
Merged build triggered.
---
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 h
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8426
---
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 enab
Github user navis closed the pull request at:
https://github.com/apache/spark/pull/8390
---
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 enabl
Github user dbtsai commented on the pull request:
https://github.com/apache/spark/pull/8426#issuecomment-134843286
LGTM. Merged into branch-1.5 and master. 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 p
Github user navis commented on the pull request:
https://github.com/apache/spark/pull/8390#issuecomment-134843288
@liancheng Sorry, char type cannot be properly compared to other text
values if once converted to string. I'll close PR.
---
If your project is set up for it, you can rep
Github user feynmanliang commented on the pull request:
https://github.com/apache/spark/pull/8451#issuecomment-134843193
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 ha
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8449#issuecomment-134842922
[Test build #41602 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41602/consoleFull)
for PR 8449 at commit
[`0e8d658`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8451#issuecomment-134841994
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8451#issuecomment-134841987
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8394#issuecomment-134840751
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8394#issuecomment-134840758
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8394#issuecomment-134840021
[Test build #41599 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41599/console)
for PR 8394 at commit
[`0fde873`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8449#issuecomment-134837723
Merged build started.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8449#issuecomment-134837672
Merged build triggered.
---
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 h
Github user yu-iskw commented on the pull request:
https://github.com/apache/spark/pull/8435#issuecomment-134836311
@mengxr All right. Thank you for letting me know.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8452#issuecomment-134836261
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8452#issuecomment-134836260
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
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/8435#issuecomment-134836198
@yu-iskw Thanks for catching it! I will include the change in #8449.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8452#issuecomment-134836165
[Test build #41597 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41597/console)
for PR 8452 at commit
[`1a26c0e`](https://github.
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8442
---
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 enab
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/8442#issuecomment-134835947
Merged into master and branch-1.5.
---
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 no
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8435
---
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 enab
Github user yu-iskw commented on the pull request:
https://github.com/apache/spark/pull/8435#issuecomment-134835735
LGTM, minus the @feynmanliang point.
Apart from this issue, it seems that there is an unnecessary comment in
PowerIterationClustering.scala. We should remove it.
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/8435#issuecomment-134835714
Merged into master and branch-1.5.
---
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 no
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8451#issuecomment-134835554
Merged build triggered.
---
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 h
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8439
---
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 enab
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8451#issuecomment-134835562
Merged build started.
---
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 mengxr commented on the pull request:
https://github.com/apache/spark/pull/8439#issuecomment-134835517
Merged into master and branch-1.5.
---
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 no
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/8435#discussion_r37947555
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModel.scala
---
@@ -46,9 +46,9 @@ import org.apache.spark.sql.{SQLContext, Row
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8445#issuecomment-134835295
[Test build #41600 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41600/consoleFull)
for PR 8445 at commit
[`24f9282`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8394#issuecomment-134835162
[Test build #41599 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41599/consoleFull)
for PR 8394 at commit
[`0fde873`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8445#issuecomment-134834703
Merged build started.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8442#issuecomment-134834698
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8442#issuecomment-134834697
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8445#issuecomment-134834688
Merged build triggered.
---
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 h
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8442#issuecomment-134834627
[Test build #41592 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41592/console)
for PR 8442 at commit
[`14700a2`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8436#issuecomment-134834323
[Test build #41598 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41598/consoleFull)
for PR 8436 at commit
[`5169ce0`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8439#issuecomment-134834264
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8394#issuecomment-134834275
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8394#issuecomment-134834283
Merged build started.
---
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 feynmanliang commented on the pull request:
https://github.com/apache/spark/pull/8435#issuecomment-134834277
LGTM overall
---
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 feat
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8439#issuecomment-134834265
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user felixcheung commented on the pull request:
https://github.com/apache/spark/pull/8360#issuecomment-134834180
ok, closing. let me know what I can help with!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If you
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8439#issuecomment-134834211
[Test build #41594 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41594/console)
for PR 8439 at commit
[`f31a148`](https://github.
Github user felixcheung closed the pull request at:
https://github.com/apache/spark/pull/8360
---
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 feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/8435#discussion_r37947104
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModel.scala
---
@@ -46,9 +46,9 @@ import org.apache.spark.sql.{SQLContex
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/8435#discussion_r37947075
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixture.scala ---
@@ -53,6 +53,7 @@ import org.apache.spark.util.Utils
* @
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8436#issuecomment-134833845
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8436#issuecomment-134833851
Merged build started.
---
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 feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/8436#discussion_r37946843
--- Diff:
mllib/src/test/java/org/apache/spark/ml/feature/JavaStopWordsRemoverSuite.java
---
@@ -0,0 +1,70 @@
+/*
+ * Licensed to the Apache Sof
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/8436#discussion_r37946831
--- Diff: docs/ml-features.md ---
@@ -306,6 +306,80 @@ regexTokenizer = RegexTokenizer(inputCol="sentence",
outputCol="words", pattern=
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/8436#discussion_r37946834
--- Diff: docs/ml-features.md ---
@@ -306,6 +306,80 @@ regexTokenizer = RegexTokenizer(inputCol="sentence",
outputCol="words", pattern=
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8451#issuecomment-134832325
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8449#issuecomment-134832339
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
1 - 100 of 834 matches
Mail list logo