Github user MLnick commented on the pull request:
https://github.com/apache/spark/pull/10607#issuecomment-196690249
@sethah given it's a copy of MLLIb impl, and given test passes and the
smaller spark-perf tests, I'm comfortable moving ahead. We should still run the
larger scale tests
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11712#issuecomment-196690170
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 projec
Github user yinxusen commented on the pull request:
https://github.com/apache/spark/pull/9#issuecomment-196690176
test it 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 featu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11712#issuecomment-196690172
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user yinxusen commented on the pull request:
https://github.com/apache/spark/pull/9#issuecomment-196690164
I'll create new JIRA on params overriding.
---
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 proj
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11712#issuecomment-196690053
Thanks - going to merge 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 hav
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/11410#issuecomment-196690093
Hi @retronym , your idea looks good, but in our situation, we only have a
class name string and it's not that easy to get the line object. BTW the REPL
classes are lo
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11712#issuecomment-196690024
**[Test build #53161 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53161/consoleFull)**
for PR 11712 at commit
[`31fc6cd`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11693#issuecomment-196689449
**[Test build #53170 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53170/consoleFull)**
for PR 11693 at commit
[`c41739c`](https://gi
Github user jerryshao commented on the pull request:
https://github.com/apache/spark/pull/11693#issuecomment-196687932
Jenkins, 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
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11697#issuecomment-196685295
I've merged this in branch-1.6.
---
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 rxin commented on the pull request:
https://github.com/apache/spark/pull/11697#issuecomment-196685350
(You'd need to close 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 th
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11696#issuecomment-196685214
@cloud-fan the logic looks good to me.
I left some comments -- try make this understandable (by adding more
comments or renaming some functions) for people that h
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11696#discussion_r56120740
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/SQLBuilder.scala ---
@@ -229,6 +224,30 @@ class SQLBuilder(logicalPlan: LogicalPlan, sqlContext:
SQ
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11696#discussion_r56120708
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/SQLBuilder.scala ---
@@ -354,59 +425,54 @@ class SQLBuilder(logicalPlan: LogicalPlan,
sqlContext: S
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11696#discussion_r56120577
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/SQLBuilder.scala ---
@@ -274,17 +293,25 @@ class SQLBuilder(logicalPlan: LogicalPlan,
sqlContext: S
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11696#discussion_r56120392
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/SQLBuilder.scala ---
@@ -316,31 +345,73 @@ class SQLBuilder(logicalPlan: LogicalPlan,
sqlContext: S
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10428#issuecomment-196683541
**[Test build #53169 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53169/consoleFull)**
for PR 10428 at commit
[`7a7a9b6`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11703#issuecomment-196683532
**[Test build #53168 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53168/consoleFull)**
for PR 11703 at commit
[`4554854`](https://gi
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11696#discussion_r56120228
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/SQLBuilder.scala ---
@@ -354,59 +425,54 @@ class SQLBuilder(logicalPlan: LogicalPlan,
sqlContext: S
Github user keypointt commented on a diff in the pull request:
https://github.com/apache/spark/pull/11703#discussion_r5612
--- Diff: project/MimaExcludes.scala ---
@@ -319,6 +319,68 @@ object MimaExcludes {
ProblemFilters.exclude[IncompatibleResultTypeProblem]("org
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11715#issuecomment-196682803
**[Test build #53167 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53167/consoleFull)**
for PR 11715 at commit
[`0d578bb`](https://gi
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r56119880
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/rules/RuleExecutor.scala
---
@@ -98,7 +107,8 @@ abstract class RuleExecutor[TreeType <: Tre
GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/11715
[SPARK-13892][SQL] remove useless lateral view handling
## What changes were proposed in this pull request?
`LATERAL VIEW` is part of the relation, and we already handled it when
convert
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/11715#issuecomment-196682504
cc @hvanhovell @rxin
---
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 th
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r56119782
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/rules/RuleExecutor.scala
---
@@ -98,7 +107,8 @@ abstract class RuleExecutor[TreeType <:
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11700
---
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 ena
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11703#discussion_r56119753
--- Diff: project/MimaExcludes.scala ---
@@ -319,6 +319,68 @@ object MimaExcludes {
ProblemFilters.exclude[IncompatibleResultTypeProblem]("org.apac
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r56119755
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/rules/RuleExecutor.scala
---
@@ -98,7 +107,8 @@ abstract class RuleExecutor[TreeType <:
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11711
---
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 ena
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11700#issuecomment-196681933
Merging in 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 project does not have t
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11708#issuecomment-196681717
cc @marmbrus
---
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
en
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11711#issuecomment-196681596
Merging in 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 project does not have t
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r56119467
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/rules/RuleExecutor.scala
---
@@ -98,7 +107,8 @@ abstract class RuleExecutor[TreeType <: Tre
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r56119479
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/BooleanSimplificationSuite.scala
---
@@ -29,6 +29,7 @@ import org.apache.spark.sq
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r56119436
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/rules/RuleExecutor.scala
---
@@ -46,15 +47,23 @@ abstract class RuleExecutor[TreeType <: Tr
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r56119362
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/rules/RuleExecutor.scala
---
@@ -98,7 +107,8 @@ abstract class RuleExecutor[TreeType <: Tre
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/11712#issuecomment-196681094
LGTM
---
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
enable
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11710
---
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 ena
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11710#issuecomment-196679919
Merging in 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 feat
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11707#issuecomment-196678560
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11707#issuecomment-196678561
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11707#issuecomment-196678381
**[Test build #53164 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53164/consoleFull)**
for PR 11707 at commit
[`139dd2a`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11710#issuecomment-196677823
**[Test build #2641 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2641/consoleFull)**
for PR 11710 at commit
[`ad073fa`](https://
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10428#issuecomment-196676937
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10428#issuecomment-196676941
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10428#issuecomment-196676928
**[Test build #53165 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53165/consoleFull)**
for PR 10428 at commit
[`cda7275`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-196676820
**[Test build #53166 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53166/consoleFull)**
for PR 11714 at commit
[`797aabb`](https://gi
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/11714
[SPARK-13891] [SQL] [TEST] Issue Exceptions when Hitting the Max Iteration
Limit in Optimizer and Analyzer
What changes were proposed in this pull request?
This PR is to issue an e
Github user retronym commented on the pull request:
https://github.com/apache/spark/pull/11410#issuecomment-196676313
Since offering review, I've found a more direct way to get to the object.
```
Welcome to Scala 2.11.8 (Java HotSpot(TM) 64-Bit Server VM, Java 1.8.0_71).
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10428#issuecomment-196676152
**[Test build #53165 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53165/consoleFull)**
for PR 10428 at commit
[`cda7275`](https://gi
Github user yinxusen commented on the pull request:
https://github.com/apache/spark/pull/11683#issuecomment-196675963
I'll add follow-up JIRA for nested pipeline support.
---
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/11410#issuecomment-196675771
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user yinxusen commented on the pull request:
https://github.com/apache/spark/pull/11683#issuecomment-196675833
@jkbradley Ready for another look.
---
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/11410#issuecomment-196675770
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11410#issuecomment-196675688
**[Test build #53157 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53157/consoleFull)**
for PR 11410 at commit
[`9be89ff`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11707#issuecomment-196675600
**[Test build #53164 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53164/consoleFull)**
for PR 11707 at commit
[`139dd2a`](https://gi
Github user GayathriMurali commented on the pull request:
https://github.com/apache/spark/pull/11707#issuecomment-196675209
@jkbradley Fixed all the minor issues and changed title. Will wait for the
tests to complete
---
If your project is set up for it, you can reply to this email
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/9#discussion_r56117376
--- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/KMeans.scala
---
@@ -258,6 +290,27 @@ class KMeans @Since("1.5.0") (
@Since("1.5.0")
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11713#issuecomment-196672117
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11713#issuecomment-196672116
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11713#issuecomment-196672054
**[Test build #53162 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53162/consoleFull)**
for PR 11713 at commit
[`8469148`](https://g
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11664
---
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 ena
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11544#issuecomment-196671690
**[Test build #53163 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53163/consoleFull)**
for PR 11544 at commit
[`5f452cc`](https://gi
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/11582#issuecomment-196671658
It looks like [https://github.com/apache/spark/pull/11707] is closer to
being merged now, so let's go with it instead. @wangmiao1981 Could you please
close this PR?
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11664#issuecomment-196671610
Merging in 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 feat
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11664#issuecomment-196671197
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11664#issuecomment-196671202
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11664#issuecomment-196670729
**[Test build #53156 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53156/consoleFull)**
for PR 11664 at commit
[`5f00d67`](https://g
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11666
---
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 ena
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11666#issuecomment-196669270
Merging in 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 project does not have t
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/11582#discussion_r56116371
--- Diff: sql/hive/pom.xml ---
@@ -38,6 +38,11 @@
+org.scala-lang
--- End diff --
Please revert the change
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11683#issuecomment-196668764
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11683#issuecomment-196668475
**[Test build #53159 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53159/consoleFull)**
for PR 11683 at commit
[`168bfc1`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11683#issuecomment-196668758
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 projec
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/11582#issuecomment-196668602
Also, to "fix the title," you'll need to put square brackets around the
JIRA: "[SPARK-13034]" and probably remove the colon ":"
---
If your project is set up for it,
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11666#issuecomment-196668356
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11666#issuecomment-196668352
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 projec
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/11582#issuecomment-196668151
Thanks both for collaborating!
@wangmiao1981 If you want to get rid of the commits merged from master,
then you can do ```git rebase upstream/master```. Feel
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11666#issuecomment-196667755
**[Test build #53155 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53155/consoleFull)**
for PR 11666 at commit
[`a47d362`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11711#issuecomment-196667168
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11711#issuecomment-196667175
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11711#issuecomment-19828
**[Test build #53160 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53160/consoleFull)**
for PR 11711 at commit
[`fbea333`](https://g
Github user HuJiayin commented on the pull request:
https://github.com/apache/spark/pull/9794#issuecomment-19064
ok!!
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11452#issuecomment-196665462
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11452#issuecomment-196665465
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11713#issuecomment-196665456
**[Test build #53162 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53162/consoleFull)**
for PR 11713 at commit
[`8469148`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11452#issuecomment-196665343
**[Test build #53153 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53153/consoleFull)**
for PR 11452 at commit
[`e9ab003`](https://g
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/9794#issuecomment-196665339
Very cool, thanks! I'll check it out.
---
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 carsonwang opened a pull request:
https://github.com/apache/spark/pull/11713
[SPARK-13889][YARN] Fix integer overflow when calculating the max number of
executor failure
## What changes were proposed in this pull request?
The max number of executor failure before fai
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11712#issuecomment-196664932
**[Test build #53161 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53161/consoleFull)**
for PR 11712 at commit
[`31fc6cd`](https://gi
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11712#issuecomment-196664585
This is based on https://github.com/apache/spark/pull/11710
For the actual diff, just look at the last commit.
---
If your project is set up for it, you can rep
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11712#issuecomment-196664605
cc @cloud-fan
---
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
e
GitHub user rxin opened a pull request:
https://github.com/apache/spark/pull/11712
[SPARK-13890][SQL] Remove some internal classes' dependency on SQLContext
## What changes were proposed in this pull request?
In general it is better for internal classes to not depend on the exter
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11711#issuecomment-196664179
**[Test build #53160 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53160/consoleFull)**
for PR 11711 at commit
[`fbea333`](https://gi
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/10242#issuecomment-196663574
Great, 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 feat
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/9#discussion_r56115607
--- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/KMeans.scala
---
@@ -258,6 +290,27 @@ class KMeans @Since("1.5.0") (
@Since("1.5.0")
GitHub user zsxwing opened a pull request:
https://github.com/apache/spark/pull/11711
[SPARK-13888][DOC]Remove Akka Receiver doc and refer to the DStream Akka
project
## What changes were proposed in this pull request?
I have copied the docs of Streaming Akka to
https://gi
Github user dbtsai commented on the pull request:
https://github.com/apache/spark/pull/11610#issuecomment-196662210
I will vote for approach 1.
SVD will be the most stable algorithm, but slowest O(mn^2 + n^3) compared
with Cholesky O(mn^2) or QR O(mn^2 - n^3/3) decomposition.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11297#issuecomment-196660958
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 projec
1 - 100 of 902 matches
Mail list logo