Skip to content

Conversation

@ScrapCodes
Copy link
Member

...'t depend on

Publish local in maven term is install

and publish otherwise is deploy

So disabled both for following projects.

@ScrapCodes
Copy link
Member Author

test this please.

@SparkQA
Copy link

SparkQA commented Sep 9, 2014

QA tests have started for PR 2329 at commit 257b79a.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 9, 2014

QA tests have finished for PR 2329 at commit 257b79a.

  • This patch fails unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@vanzin
Copy link
Contributor

vanzin commented Sep 9, 2014

Man, I really dislike how verbose maven can be... ()

Anyway... LGTM.

@ScrapCodes
Copy link
Member Author

Jenkins, retest this please.

@pwendell
Copy link
Contributor

Jenkins, retest this please.

@pwendell
Copy link
Contributor

LGTM as well pending tests. Thanks @ScrapCodes

@SparkQA
Copy link

SparkQA commented Sep 12, 2014

QA tests have started for PR 2329 at commit 257b79a.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 12, 2014

QA tests have finished for PR 2329 at commit 257b79a.

  • This patch passes unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@asfgit asfgit closed this in f493f79 Sep 15, 2014
@ScrapCodes ScrapCodes deleted the SPARK-3452/maven-skip-install branch June 3, 2015 06:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants