Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/9669#issuecomment-156211695
No surprises. Exposed a lot of test failures. Fixing them
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9673#issuecomment-156217392
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9672#issuecomment-156217436
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9673#issuecomment-156217437
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9673#issuecomment-156224755
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
GitHub user andrewor14 opened a pull request:
https://github.com/apache/spark/pull/9676
[SPARK-11710] Document new memory management model
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/andrewor14/spark memory-management-docs
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9667#issuecomment-156227937
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9668#issuecomment-156227938
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9676#issuecomment-156227970
**[Test build #45766 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45766/consoleFull)**
for PR 9676 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9667#issuecomment-156227909
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9668#issuecomment-156227907
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9668#issuecomment-156230613
**[Test build #45768 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45768/consoleFull)**
for PR 9668 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9677#issuecomment-156236667
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
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/9677#issuecomment-156236535
test this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9678#issuecomment-156236638
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9678#issuecomment-15623
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9642#issuecomment-156243899
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9642#issuecomment-156243874
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9669#issuecomment-156243648
**[Test build #45776 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45776/consoleFull)**
for PR 9669 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9642#issuecomment-156246436
**[Test build #45777 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45777/consoleFull)**
for PR 9642 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9664
---
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 marmbrus commented on the pull request:
https://github.com/apache/spark/pull/9646#issuecomment-156225043
This seems fine for now, but its not clear to me why the right answer isn't
to read it in as a higher precision signed type.
---
If your project is set up for it, you
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/9646#issuecomment-156225166
Merging to master and 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 have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9676#issuecomment-156232980
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/9664#issuecomment-156233129
+1
---
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 marmbrus commented on the pull request:
https://github.com/apache/spark/pull/9642#issuecomment-156233889
ok to test
---
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 marmbrus commented on the pull request:
https://github.com/apache/spark/pull/9385#issuecomment-156238962
I'm sorry, I don't see where this `expand()` function you are talking about
is, or why it should not use `semanticEquals`. The whole point of
`semanticEquals` is "are
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9669#issuecomment-156244305
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9669#issuecomment-156244297
**[Test build #45776 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45776/consoleFull)**
for PR 9669 at commit
Github user squito commented on the pull request:
https://github.com/apache/spark/pull/9610#issuecomment-156244415
you can take this test case if you like:
https://github.com/squito/spark/blob/SPARK-8029_first_wins/core/src/test/scala/org/apache/spark/ShuffleSuite.scala#L351
Github user tnachen commented on the pull request:
https://github.com/apache/spark/pull/9637#issuecomment-156244571
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
enabled
Github user tnachen commented on the pull request:
https://github.com/apache/spark/pull/9637#issuecomment-156244521
I think this is fine for now, I was thinking leaving some comments about
how it should be launched with Marathon but I think later we can add a example
json in the
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9380
---
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 JihongMA commented on the pull request:
https://github.com/apache/spark/pull/9380#issuecomment-156249080
@mengxr sure, will take care mean via seperate 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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9667#issuecomment-156211034
**[Test build #45760 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45760/consoleFull)**
for PR 9667 at commit
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/9673#issuecomment-156217004
/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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9666
---
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 pull request:
https://github.com/apache/spark/pull/9674#issuecomment-156221746
**[Test build #45764 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45764/consoleFull)**
for PR 9674 at commit
Github user tylerprete closed the pull request at:
https://github.com/apache/spark/pull/8992
---
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 tylerprete commented on the pull request:
https://github.com/apache/spark/pull/8992#issuecomment-156224009
Yeah, sorry, life got busy and haven't made progress on this.
Realistically, I'm not sure I'll get to it right away. If I ever get around
to making it work,
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/9297#discussion_r44711788
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SQLExecution.scala ---
@@ -45,25 +46,14 @@ private[sql] object SQLExecution {
GitHub user jkbradley opened a pull request:
https://github.com/apache/spark/pull/9674
[SPARK-11612] [ML] Pipeline and PipelineModel persistence
Pipeline and PipelineModel extend Readable and Writable. Persistence
succeeds only when all stages are Writable.
Note: This PR
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9637#issuecomment-156229506
**[Test build #45767 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45767/consoleFull)**
for PR 9637 at commit
GitHub user mengxr opened a pull request:
https://github.com/apache/spark/pull/9677
[WIP] [SPARK-11672] flaky spark.ml read/write tests
Add callSite info for debugging. I will run Jenkins several times.
You can merge this pull request into a Git repository by running:
$ git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9642#issuecomment-156235032
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9678#issuecomment-156234993
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
GitHub user jkbradley opened a pull request:
https://github.com/apache/spark/pull/9678
[SPARK-11712] [ML] Make spark.ml LDAModel be abstract
Per discussion in the initial Pipelines LDA PR
[https://github.com/apache/spark/pull/9513], we should make LDAModel abstract
and create a
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9673#issuecomment-156238595
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9673#issuecomment-156238498
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9669#issuecomment-156240672
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9678#issuecomment-156240853
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9669#issuecomment-156240715
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9673#issuecomment-156240423
**[Test build #45775 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45775/consoleFull)**
for PR 9673 at commit
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/9678#discussion_r44720208
--- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/LDA.scala ---
@@ -468,7 +452,37 @@ class LDAModel private[ml] (
/**
* ::
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9373#issuecomment-156255395
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9671#issuecomment-156212136
[Test build #45756 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45756/console)
for PR 9671 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9671#issuecomment-156212227
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9636#issuecomment-156214399
**[Test build #45751 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45751/consoleFull)**
for PR 9636 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9636#issuecomment-156214620
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9636#issuecomment-156214616
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
GitHub user marmbrus opened a pull request:
https://github.com/apache/spark/pull/9673
[SPARK-11654][SQL] add reduce to GroupedDataset
This PR adds a new method, `reduce`, to `GroupedDataset`, which allows
similar operations to `reduceByKey` on a traditional `PairRDD`.
GitHub user tedyu opened a pull request:
https://github.com/apache/spark/pull/9672
[SPARK-11699] TrackStateRDDSuite fails on Jenkins builds
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/tedyu/spark master
Alternatively you
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/9664#issuecomment-156218775
This LGTM, but what about functions that are added to the metastore?
Should we also check the metadata hive next?
Eitherway, I think we can merge this now to
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9673#issuecomment-156218673
**[Test build #45762 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45762/consoleFull)**
for PR 9673 at commit
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/9637#discussion_r44710758
--- Diff: docs/job-scheduling.md ---
@@ -56,36 +56,31 @@ provide another approach to share RDDs.
## Dynamic Resource Allocation
-Spark
Github user NarineK commented on the pull request:
https://github.com/apache/spark/pull/9667#issuecomment-156228151
It seems that there is a problem in java for many output parameters.
Introducing a Result output object would be probably better.
---
If your project is set up
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9667#issuecomment-156232189
**[Test build #45769 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45769/consoleFull)**
for PR 9667 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9667#issuecomment-156232202
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9667#issuecomment-156232201
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
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/9664#issuecomment-156232331
@marmbrus Hive UDFs stored in metastore is a harder problem. For those
functions, we need to talk to metadataHive. But, when we do function lookup, we
do not know which
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9637#issuecomment-156233547
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9637#issuecomment-156233546
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9637#issuecomment-156233387
**[Test build #45767 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45767/consoleFull)**
for PR 9637 at commit
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/9664#discussion_r44714829
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala ---
@@ -454,7 +454,7 @@ class HiveContext private[hive](
// Note that
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9380#issuecomment-156234586
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
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/9588#issuecomment-156236418
I'm not sure we should support the exact semantics of hives temporary
tables. I'd be more inclined to intercept such DDL and convert it to a Spark
SQL temporary table
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9678#issuecomment-156246240
**[Test build #2051 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2051/consoleFull)**
for PR 9678 at commit
Github user feynmanliang commented on the pull request:
https://github.com/apache/spark/pull/9678#issuecomment-156249049
LGTM, all comments minor / optional
---
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/9672#issuecomment-156217396
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
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/9655#issuecomment-156223468
I'm inclined to agree with Sean, this seems like code churn for the sake
of code churn.
---
If your project is set up for it, you can reply to this email and have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9675#issuecomment-156223456
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9675#issuecomment-156223424
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
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/9610#discussion_r44715168
--- Diff:
core/src/main/scala/org/apache/spark/shuffle/IndexShuffleBlockResolver.scala ---
@@ -93,6 +93,29 @@ private[spark] class
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9642#issuecomment-156235000
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
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/9610#discussion_r44715331
--- Diff:
core/src/main/scala/org/apache/spark/shuffle/IndexShuffleBlockResolver.scala ---
@@ -93,6 +93,29 @@ private[spark] class
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9678#issuecomment-156235031
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9677#issuecomment-156236641
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
Github user davis commented on the pull request:
https://github.com/apache/spark/pull/9536#issuecomment-156239448
@davies
---
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 pull request:
https://github.com/apache/spark/pull/9642#issuecomment-156239366
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9642#issuecomment-156239352
**[Test build #45772 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45772/consoleFull)**
for PR 9642 at commit
Github user tedyu closed the pull request at:
https://github.com/apache/spark/pull/9672
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9636#issuecomment-156247885
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
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/9380#issuecomment-156247920
LGTM. Merged into master and branch-1.6. Thanks! Btw, there is a minor
style issue I marked inline. @JihongMA Could you submit another PR to change
the output of
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/9678#discussion_r44720409
--- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/LDA.scala ---
@@ -314,31 +314,31 @@ private[clustering] trait LDAParams extends Params
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9636#issuecomment-156251056
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9636#issuecomment-156251082
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
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/9636#issuecomment-156215451
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
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/9666#issuecomment-156221680
Thanks - I've merged 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
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/9513#issuecomment-156221541
Hm, good point. OK I'll try that & ping you on the PR.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9668#issuecomment-156223722
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
1 - 100 of 1100 matches
Mail list logo