Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2219#issuecomment-53979567
[QA tests have
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19534/consoleFull)
for PR 2219 at commit
[`613b021`](https://github.com/ap
GitHub user velvia opened a pull request:
https://github.com/apache/spark/pull/2219
Add a Community Projects page
This adds a new page to the docs listing community projects -- those
created outside of Apache Spark that are of interest to the community of Spark
users. Anybody can
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/2144#issuecomment-53979487
Yes, I will do that next week.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/60#issuecomment-53979384
[QA tests have
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19533/consoleFull)
for PR 60 at commit
[`27df6ce`](https://github.com/apache
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1992#issuecomment-53978638
[QA tests have
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19532/consoleFull)
for PR 1992 at commit
[`b2a044a`](https://github.com/ap
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2178#issuecomment-53978615
[QA tests have
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19531/consoleFull)
for PR 2178 at commit
[`1445ef2`](https://github.com/a
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/1992#issuecomment-53978586
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 ha
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/1992#issuecomment-53978314
hi @pwendell, jenkins fetch error.
---
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/2178#issuecomment-53978226
@pwendell I think some Spark SQL tests are failing. Spark SQL isn't
completely compatible with sort based shuffle yet.
---
If your project is set up for it, you can reply
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2178#issuecomment-53978007
[QA tests have
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19531/consoleFull)
for PR 2178 at commit
[`1445ef2`](https://github.com/ap
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/2102#issuecomment-53977986
Can you create a JIRA for this issue? I'm going to re-word this a bit when
I merge it, I think it's fine to say that certain users have reported issues
building behind p
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/2178#issuecomment-53977954
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 have
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/2014#issuecomment-53977940
Made a few comments inline. On building docs, my favorite idea is just to
have the README link to the upstream docs, and then change the upstream docs to
be called "Buil
Github user pwendell commented on a diff in the pull request:
https://github.com/apache/spark/pull/2014#discussion_r16933038
--- Diff: README.md ---
@@ -66,78 +69,24 @@ Many of the example programs print usage help if no
params are given.
## Running Tests
-Test
Github user pwendell commented on a diff in the pull request:
https://github.com/apache/spark/pull/2014#discussion_r16933034
--- Diff: README.md ---
@@ -66,78 +69,24 @@ Many of the example programs print usage help if no
params are given.
## Running Tests
-Test
Github user pwendell commented on a diff in the pull request:
https://github.com/apache/spark/pull/2014#discussion_r16933031
--- Diff: CONTRIBUTING.md ---
@@ -0,0 +1,12 @@
+## Contributing to Spark
--- End diff --
Yeah, seems fine to have this here. It might make it
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/2212#issuecomment-53977840
Can you add `[YARN]` to the title so that it gets sorted properly? Thanks
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/2176#issuecomment-53977827
I added a comment about the experimental formatting
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If yo
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/2217#issuecomment-53977801
Hi there,
The immutability of broadcast variables might be assumed in other places in
the code base. Since this approach requires re-broadcasting the entire cont
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/2184
---
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 pwendell commented on the pull request:
https://github.com/apache/spark/pull/2184#issuecomment-53977717
Cool - thanks Nick!
---
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 fe
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/1992#issuecomment-53977588
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 have
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/1698
---
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/254
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/1356
---
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/1922
---
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/2135
---
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 uncleGen commented on the pull request:
https://github.com/apache/spark/pull/1356#issuecomment-53977557
okay!
---
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 scwf commented on the pull request:
https://github.com/apache/spark/pull/1992#issuecomment-53977519
@mateiz retest this again, tests failed in sparkstreaming, thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1992#issuecomment-53977395
[QA tests have
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19529/consoleFull)
for PR 1992 at commit
[`b2a044a`](https://github.com/a
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/1356#issuecomment-53977396
Okay I think this is no longer necessary now that we fixed the issue
causing lag in processing events. So I'd like to close this issue for now.
---
If your project is s
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/2218#issuecomment-53977030
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 pro
GitHub user kramimus opened a pull request:
https://github.com/apache/spark/pull/2218
[SPARK-2558][DOCS] Add spark.yarn.queue description to YARN doc
Put original YARN queue spark-submit arg description in
running-on-yarn html table and example command line
You can merge this pu
Github user joesu commented on the pull request:
https://github.com/apache/spark/pull/1737#issuecomment-53976828
Another way is to include max length information in the BinaryType type,
just like the FixedLenByteArray type in this pull request. Thus we can maintain
only one binary dat
Github user joesu commented on the pull request:
https://github.com/apache/spark/pull/1737#issuecomment-53976708
It's not that straightforward to reuse BinaryType for handling parquet's
binary type and fixed_len_byte_array types because these two types are
incompatible in the parquet
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1778#issuecomment-53976440
[QA tests have
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19528/consoleFull)
for PR 1778 at commit
[`75a0b51`](https://github.com/a
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1778#issuecomment-53976161
[QA tests have
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19527/consoleFull)
for PR 1778 at commit
[`0f12ade`](https://github.com/a
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1992#issuecomment-53976069
[QA tests have
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19529/consoleFull)
for PR 1992 at commit
[`b2a044a`](https://github.com/ap
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/1992#issuecomment-53976026
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 have thi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1778#issuecomment-53975553
[QA tests have
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19528/consoleFull)
for PR 1778 at commit
[`75a0b51`](https://github.com/ap
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/1992#issuecomment-53975519
@mateiz , retest this please, tests failed due to forked process exit code
is not zero.
https://github.com/apache/spark/pull/2108
---
If your project is set up for it,
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/2209#issuecomment-53975391
Sounds good. I was not sure how to correctly query those results with
ambiguous schemas when I added that check. Seems an more informative logging
entry is better than an e
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1778#issuecomment-53975264
[QA tests have
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19527/consoleFull)
for PR 1778 at commit
[`0f12ade`](https://github.com/ap
Github user rezazadeh commented on the pull request:
https://github.com/apache/spark/pull/1778#issuecomment-53975250
Style changes made. Experimental results below.
We run DIMSUM daily on a production-scale ads dataset. After replacing the
traditional cosine similarity computa
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/2209#issuecomment-53975089
I actually encountered the error with a jsonRDD, but yeah it could happen
with parquet files as well. Your comment about joins though makes me think
that we should just
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/2215#discussion_r16932497
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/commands.scala ---
@@ -90,10 +90,9 @@ case class SetCommand(
throw new IllegalArg
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2210#issuecomment-53975007
[QA tests have
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19526/consoleFull)
for PR 2210 at commit
[`a25d27a`](https://github.com/a
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1992#issuecomment-53974654
[QA tests have
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19525/consoleFull)
for PR 1992 at commit
[`b2a044a`](https://github.com/a
Github user manishamde commented on the pull request:
https://github.com/apache/spark/pull/2125#issuecomment-53974143
Apart from the discussion around the correct place for centriod
calculations and some minor code style comments, it looks good to me. If it's
too much work to change i
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/2210
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2210#issuecomment-53974083
[QA tests have
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19526/consoleFull)
for PR 2210 at commit
[`a25d27a`](https://github.com/ap
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/2210#issuecomment-53974057
Alright, 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
Github user holdenk commented on the pull request:
https://github.com/apache/spark/pull/2210#issuecomment-53974032
@mateiz Thanks, completely forgot to check the javadoc.
---
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 mateiz commented on the pull request:
https://github.com/apache/spark/pull/2210#issuecomment-53973967
(And please add [SPARK-3318] at the beginning of your PR title)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as w
Github user manishamde commented on a diff in the pull request:
https://github.com/apache/spark/pull/2125#discussion_r16932280
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/tree/impl/DTStatsAggregator.scala
---
@@ -0,0 +1,208 @@
+/*
+ * Licensed to the Apache Softw
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/2210#issuecomment-53973932
Actually you missed JavaSparkContext; it has the same issue
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/2210#issuecomment-53973919
Looks good, 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 naftaliharris commented on the pull request:
https://github.com/apache/spark/pull/1057#issuecomment-53973576
@mateiz oh yeah, no problem. Thanks again for the fixes!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as w
Github user naftaliharris closed the pull request at:
https://github.com/apache/spark/pull/1057
---
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
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/85#issuecomment-53973578
Yeah up to you, you should either update it or close the PR if you think
everything is there already.
---
If your project is set up for it, you can reply to this email and
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/915#issuecomment-53973561
Alright, thanks for taking a look at 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 d
Github user mateiz commented on a diff in the pull request:
https://github.com/apache/spark/pull/2118#discussion_r16932221
--- Diff:
core/src/main/scala/org/apache/spark/input/EscapedTextInputFormat.scala ---
@@ -0,0 +1,236 @@
+/*
+ * Licensed to the Apache Software Foundat
Github user mateiz commented on a diff in the pull request:
https://github.com/apache/spark/pull/2118#discussion_r16932218
--- Diff:
core/src/main/scala/org/apache/spark/input/EscapedTextInputFormat.scala ---
@@ -0,0 +1,236 @@
+/*
+ * Licensed to the Apache Software Foundat
Github user rcompton closed the pull request at:
https://github.com/apache/spark/pull/915
---
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 rcompton commented on the pull request:
https://github.com/apache/spark/pull/915#issuecomment-53973422
@mateiz no, for the reasons mentioned by Sean as well as the new work by
Sigmoid, you don't need this patch.
---
If your project is set up for it, you can reply to thi
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/1121#issuecomment-53973370
@YanjieGao do you see a major tradeoff in compressed size and speed with
this codec over our current ones?
Also, I'm not sure your patch will compile as written. T
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/915#issuecomment-53973167
@rcompton I believe this has been addressed by Sigmoid's recent work for
Pig on Spark: https://issues.apache.org/jira/browse/PIG-4059. Given that, do we
still need this pat
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/1388#issuecomment-53973123
@sryza just wondering, will you have time to update this for Python? As I
said it would be useful to include.
---
If your project is set up for it, you can reply to this
Github user CodingCat commented on the pull request:
https://github.com/apache/spark/pull/85#issuecomment-53973073
sure, because other people told me some of the parameters are not supposed
to be configurableso I pend the work hereI can go through it again to
check the missing
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/89#issuecomment-53973053
I agree, we should not expose these to the user given the recent changes.
Would it be okay to close this PR?
---
If your project is set up for it, you can reply to this ema
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/85#issuecomment-53973039
@CodingCat are you still working on this patch? The doc page changed
significantly in 1.0, so maybe a lot of this info is still in, but it would be
good to look over it and
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1992#issuecomment-53973019
[QA tests have
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19525/consoleFull)
for PR 1992 at commit
[`b2a044a`](https://github.com/ap
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/1057#issuecomment-53972967
Hey @naftaliharris, might closing this pull request now that this has been
fixed in other PRs?
---
If your project is set up for it, you can reply to this email and have
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/1992#issuecomment-53972940
Looks good to me pending tests
---
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 mateiz commented on the pull request:
https://github.com/apache/spark/pull/1992#issuecomment-53972938
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 have thi
Github user mateiz commented on a diff in the pull request:
https://github.com/apache/spark/pull/2211#discussion_r16932123
--- Diff: repl/src/main/scala/org/apache/spark/repl/SparkILoop.scala ---
@@ -965,11 +966,9 @@ class SparkILoop(in0: Option[BufferedReader],
protected val out:
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/1843
---
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 mateiz commented on the pull request:
https://github.com/apache/spark/pull/1843#issuecomment-53971505
Thanks Marcelo! I've merged this in.
---
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 n
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2184#issuecomment-53971091
[QA tests have
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19524/consoleFull)
for PR 2184 at commit
[`33786ac`](https://github.com/a
Github user manishamde commented on the pull request:
https://github.com/apache/spark/pull/2125#issuecomment-53971034
The ordered categorical features are not binned and the centriods are
re-calculated using the entire bin aggregate every level. I can see the
improvement in accuracy h
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/2144#issuecomment-53970492
@davies just curious, do all the unit tests run if you do `run-tests` with
`pypy`? We should make sure they do, and add a command in there to test this in
Jenkins (ask Pat
Github user manishamde commented on a diff in the pull request:
https://github.com/apache/spark/pull/2125#discussion_r16931670
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/tree/impl/DTStatsAggregator.scala
---
@@ -0,0 +1,208 @@
+/*
+ * Licensed to the Apache Softw
Github user manishamde commented on a diff in the pull request:
https://github.com/apache/spark/pull/2125#discussion_r16931674
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/tree/impl/DTStatsAggregator.scala
---
@@ -0,0 +1,208 @@
+/*
+ * Licensed to the Apache Softw
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2184#issuecomment-53969763
[QA tests have
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19524/consoleFull)
for PR 2184 at commit
[`33786ac`](https://github.com/ap
Github user nchammas commented on the pull request:
https://github.com/apache/spark/pull/2184#issuecomment-53969665
@pwendell I think we're all set now.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2214#issuecomment-53969391
[QA tests have
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19523/consoleFull)
for PR 2214 at commit
[`983d030`](https://github.com/a
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2184#issuecomment-53968973
[QA tests have
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19522/consoleFull)
for PR 2184 at commit
[`638c0e4`](https://github.com/a
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/2195#issuecomment-53967657
ok to test 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/2217#issuecomment-53967605
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 pro
GitHub user viirya opened a pull request:
https://github.com/apache/spark/pull/2217
[SPARK-3327] Make broadcasted value mutable for caching useful information
This PR makes broadcasted value mutable for caching useful information when
implementing some algorithms that iteratively ru
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2214#issuecomment-53967407
[QA tests have
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19523/consoleFull)
for PR 2214 at commit
[`983d030`](https://github.com/ap
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/2214#issuecomment-53967270
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 feat
Github user nchammas commented on the pull request:
https://github.com/apache/spark/pull/2184#issuecomment-53966890
Hmm, looks like I need to fix something now that this doesn't merge cleanly
anymore. Investigating.
---
If your project is set up for it, you can reply to this email an
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2184#issuecomment-53966846
[QA tests have
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19522/consoleFull)
for PR 2184 at commit
[`638c0e4`](https://github.com/ap
Github user nchammas commented on a diff in the pull request:
https://github.com/apache/spark/pull/2184#discussion_r16930998
--- Diff: dev/run-tests-jenkins ---
@@ -138,7 +141,7 @@ function post_message () {
test_result="$?"
if [ "$test_result" -eq "124" ]; then
Github user pwendell commented on a diff in the pull request:
https://github.com/apache/spark/pull/2184#discussion_r16930734
--- Diff: dev/run-tests-jenkins ---
@@ -138,7 +141,7 @@ function post_message () {
test_result="$?"
if [ "$test_result" -eq "124" ]; then
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/2184#issuecomment-53965382
Minor style note, but otherwise 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
Github user pwendell commented on a diff in the pull request:
https://github.com/apache/spark/pull/2176#discussion_r16930621
--- Diff: core/src/main/scala/org/apache/spark/api/java/JavaRDDLike.scala
---
@@ -574,4 +574,15 @@ trait JavaRDDLike[T, This <: JavaRDDLike[T, This]]
extend
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2176#issuecomment-53961548
[QA tests have
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19521/consoleFull)
for PR 2176 at commit
[`5536d55`](https://github.com/a
Github user lirui-intel commented on the pull request:
https://github.com/apache/spark/pull/2176#issuecomment-53960113
Thanks @rxin , @vanzin for the review. I've added experimental mark in the
java doc. I see that mima can automatically exclude DeveloperApi and
Experimental classes,
1 - 100 of 112 matches
Mail list logo