You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by pwendell <gi...@git.apache.org> on 2015/01/16 23:34:47 UTC

[GitHub] spark pull request: [SPARK-5289]: Backport publishing of repl, yar...

GitHub user pwendell opened a pull request:

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

    [SPARK-5289]: Backport publishing of repl, yarn into branch-1.2.

    This change was done in SPARK-4048 as part of a larger refactoring,
    but we need to backport this publishing of yarn and repl into Spark
    1.2, so that we can cut a 1.2.1 release with these artifacts.

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

    $ git pull https://github.com/pwendell/spark skip-deps

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

    https://github.com/apache/spark/pull/4079.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 #4079
    
----
commit 807b833680d433ada6f9fd0e262197ffa8de5f89
Author: Patrick Wendell <pa...@databricks.com>
Date:   2015-01-16T22:31:56Z

    [SPARK-5289]: Backport publishing of repl, yarn into branch-1.2.
    
    This change was done in SPARK-4048 as part of a larger refactoring,
    but we need to backport this publishing of yarn and repl into Spark
    1.2, so that we can cut a 1.2.1 release with these artifacts.

----


---
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-5289]: Backport publishing of repl, yar...

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

    https://github.com/apache/spark/pull/4079#issuecomment-70391965
  
    Okay I'm pulling these two in. I think we're all on the same page.


---
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-5289]: Backport publishing of repl, yar...

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

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


---
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-5289]: Backport publishing of repl, yar...

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

    https://github.com/apache/spark/pull/4079#issuecomment-70344532
  
    Test PASSed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25684/
    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-5289]: Backport publishing of repl, yar...

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

    https://github.com/apache/spark/pull/4079#issuecomment-70344525
  
      [Test build #25684 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25684/consoleFull) for   PR 4079 at commit [`807b833`](https://github.com/apache/spark/commit/807b833680d433ada6f9fd0e262197ffa8de5f89).
     * 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-5289]: Backport publishing of repl, yar...

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

    https://github.com/apache/spark/pull/4079#issuecomment-70334130
  
      [Test build #25684 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25684/consoleFull) for   PR 4079 at commit [`807b833`](https://github.com/apache/spark/commit/807b833680d433ada6f9fd0e262197ffa8de5f89).
     * 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-5289]: Backport publishing of repl, yar...

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

    https://github.com/apache/spark/pull/4079#issuecomment-70390243
  
    @vanzin I tried to cover it in #4080 - but basically there were changes you made that were anyways being requested by others in the community (asking us to publish these to Maven, which we did prior to 1.2).


---
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-5289]: Backport publishing of repl, yar...

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

    https://github.com/apache/spark/pull/4079#issuecomment-70339542
  
    Hi @pwendell , just want to make sure you're doing this for the right reasons. The change I made in SPARK-4048 wasn't so that these artifacts would be published, but because that enables running specific modules in the maven build directly (instead of running the whole build just to re-build a specific module).


---
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-5289]: Backport publishing of repl, yar...

Posted by pwendell <gi...@git.apache.org>.
GitHub user pwendell reopened a pull request:

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

    [SPARK-5289]: Backport publishing of repl, yarn into branch-1.2.

    This change was done in SPARK-4048 as part of a larger refactoring,
    but we need to backport this publishing of yarn and repl into Spark
    1.2, so that we can cut a 1.2.1 release with these artifacts.

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

    $ git pull https://github.com/pwendell/spark skip-deps

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

    https://github.com/apache/spark/pull/4079.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 #4079
    
----
commit 807b833680d433ada6f9fd0e262197ffa8de5f89
Author: Patrick Wendell <pa...@databricks.com>
Date:   2015-01-16T22:31:56Z

    [SPARK-5289]: Backport publishing of repl, yarn into branch-1.2.
    
    This change was done in SPARK-4048 as part of a larger refactoring,
    but we need to backport this publishing of yarn and repl into Spark
    1.2, so that we can cut a 1.2.1 release with these artifacts.

----


---
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-5289]: Backport publishing of repl, yar...

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

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


---
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