You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by brkyvz <gi...@git.apache.org> on 2015/02/17 22:15:51 UTC

[GitHub] spark pull request: [SPARK-5811] Added documentation for maven coo...

GitHub user brkyvz opened a pull request:

    https://github.com/apache/spark/pull/4662

    [SPARK-5811] Added documentation for maven coordinates and added Spark Packages support

    Documentation for maven coordinates + Spark Package support. Added pyspark tests for `--packages`

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/brkyvz/spark SPARK-5811

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/4662.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #4662
    
----
commit c60156dd67900db25f7071642f96afde9461c8c7
Author: Burak Yavuz <br...@gmail.com>
Date:   2015-02-14T19:08:52Z

    [SPARK-5811] Added documentation for maven coordinates

commit a3eb7177c5dbb6ed9eb1919fda5e29f9d90c52b0
Author: Burak Yavuz <br...@gmail.com>
Date:   2015-02-17T06:50:46Z

    Merge branch 'master' of github.com:apache/spark into SPARK-5811

commit 17d3f76fa4a542a0e47fc60579b17409ebeebc7d
Author: Davies Liu <da...@databricks.com>
Date:   2015-02-17T18:09:21Z

    support .jar as python package

commit 560d13bee3be24104d2725a31e8036d9b8448e08
Author: Burak Yavuz <br...@gmail.com>
Date:   2015-02-17T18:47:06Z

    before PR

commit 25c9b9fa79eca5fe5cd9ed3d4c4c558dc9dd16ce
Author: Burak Yavuz <br...@gmail.com>
Date:   2015-02-17T18:50:32Z

    Merge branch 'master' of github.com:apache/spark into python-jar

commit 8fb02e5f198b82b90472bd18b826bb6a1cd33fe5
Author: Burak Yavuz <br...@gmail.com>
Date:   2015-02-17T20:48:54Z

    merged master

commit 4ef4046261fe43f59523f3fb84c593b7ebefe3e8
Author: Burak Yavuz <br...@gmail.com>
Date:   2015-02-17T21:05:01Z

    ready for PR

commit a8bd6b789e3f37a38d0fa9456d9d4c30b1a8d56d
Author: Burak Yavuz <br...@gmail.com>
Date:   2015-02-17T21:13:04Z

    submit 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 your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-5811] Added documentation for maven coo...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/4662#issuecomment-74770786
  
    Test FAILed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27656/
    Test FAILed.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-5811] Added documentation for maven coo...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the pull request:

    https://github.com/apache/spark/pull/4662#issuecomment-74770780
  
      [Test build #27656 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/27656/consoleFull) for   PR 4662 at commit [`64cb8ee`](https://github.com/apache/spark/commit/64cb8ee191ffc3059f8d43222bee893e87a022bf).
     * This patch **fails Spark unit tests**.
     * This patch merges cleanly.
     * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-5811] Added documentation for maven coo...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/4662#issuecomment-74758695
  
    Test FAILed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27655/
    Test FAILed.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-5811] Added documentation for maven coo...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/4662#issuecomment-74756037
  
    Test FAILed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27654/
    Test FAILed.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-5811] Added documentation for maven coo...

Posted by pwendell <gi...@git.apache.org>.
Github user pwendell commented on the pull request:

    https://github.com/apache/spark/pull/4662#issuecomment-74791922
  
    Hey @brkyvz thanks for this, I'm going to merge it. I might remove Spark Packages-specific comments. While it's great that this will work well with Spark packages, I think we want this to be a fairly neutral thing where it can be used with other maven repositories as well.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-5811] Added documentation for maven coo...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/4662#issuecomment-74783079
  
    Test PASSed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27659/
    Test PASSed.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-5811] Added documentation for maven coo...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the pull request:

    https://github.com/apache/spark/pull/4662#issuecomment-74755784
  
      [Test build #27654 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/27654/consoleFull) for   PR 4662 at commit [`a8bd6b7`](https://github.com/apache/spark/commit/a8bd6b789e3f37a38d0fa9456d9d4c30b1a8d56d).
     * This patch merges cleanly.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-5811] Added documentation for maven coo...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the pull request:

    https://github.com/apache/spark/pull/4662#issuecomment-74771232
  
      [Test build #27659 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/27659/consoleFull) for   PR 4662 at commit [`56ccccd`](https://github.com/apache/spark/commit/56ccccd2b9d765345c4ec71fb4492f7de7e90784).
     * This patch merges cleanly.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-5811] Added documentation for maven coo...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the pull request:

    https://github.com/apache/spark/pull/4662#issuecomment-74760331
  
      [Test build #27656 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/27656/consoleFull) for   PR 4662 at commit [`64cb8ee`](https://github.com/apache/spark/commit/64cb8ee191ffc3059f8d43222bee893e87a022bf).
     * This patch merges cleanly.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-5811] Added documentation for maven coo...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the pull request:

    https://github.com/apache/spark/pull/4662#issuecomment-74758524
  
      [Test build #27655 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/27655/consoleFull) for   PR 4662 at commit [`c07b81e`](https://github.com/apache/spark/commit/c07b81e0be3befc0c5ebf7eaee3f640eb904afc5).
     * This patch merges cleanly.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-5811] Added documentation for maven coo...

Posted by brkyvz <gi...@git.apache.org>.
Github user brkyvz commented on the pull request:

    https://github.com/apache/spark/pull/4662#issuecomment-74770805
  
    Adding spark-packages as a repo broke one test. Pushing update once I make sure it passes it locally.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-5811] Added documentation for maven coo...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the pull request:

    https://github.com/apache/spark/pull/4662#issuecomment-74783070
  
      [Test build #27659 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/27659/consoleFull) for   PR 4662 at commit [`56ccccd`](https://github.com/apache/spark/commit/56ccccd2b9d765345c4ec71fb4492f7de7e90784).
     * This patch **passes all tests**.
     * This patch merges cleanly.
     * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-5811] Added documentation for maven coo...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the pull request:

    https://github.com/apache/spark/pull/4662#issuecomment-74756031
  
      [Test build #27654 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/27654/consoleFull) for   PR 4662 at commit [`a8bd6b7`](https://github.com/apache/spark/commit/a8bd6b789e3f37a38d0fa9456d9d4c30b1a8d56d).
     * This patch **fails Python style tests**.
     * This patch merges cleanly.
     * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-5811] Added documentation for maven coo...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/spark/pull/4662


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-5811] Added documentation for maven coo...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the pull request:

    https://github.com/apache/spark/pull/4662#issuecomment-74758693
  
      [Test build #27655 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/27655/consoleFull) for   PR 4662 at commit [`c07b81e`](https://github.com/apache/spark/commit/c07b81e0be3befc0c5ebf7eaee3f640eb904afc5).
     * This patch **fails Python style tests**.
     * This patch merges cleanly.
     * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org