Skip to content

Conversation

@vanzin
Copy link
Contributor

@vanzin vanzin commented Sep 2, 2015

This avoids them being mistakenly pulled instead of the newer ones that
Spark actually uses. Spark only depends on these artifacts transitively,
so sometimes maven just decides to pick tachyon's version of the
dependency for whatever reason.

Marcelo Vanzin added 2 commits September 2, 2015 11:36
This avoids it being mistakenly pulled instead of the newer one that
Spark actually uses. Spark only depends on curator-framework transitively,
so sometimes maven just decides to pick tachyon's version of that
dependency for whatever reason.
@vanzin vanzin changed the title [SPARK-10421] [build] Exclude curator-framework from tachyon dependency. [SPARK-10421] [build] Exclude curator artifacts from tachyon dependencies. Sep 2, 2015
@SparkQA
Copy link

SparkQA commented Sep 2, 2015

Test build #41940 has finished for PR 8577 at commit a320d1b.

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

@SparkQA
Copy link

SparkQA commented Sep 2, 2015

Test build #41938 has finished for PR 8577 at commit 8d8683c.

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

@srowen
Copy link
Member

srowen commented Sep 2, 2015

LGTM.

@vanzin
Copy link
Contributor Author

vanzin commented Sep 3, 2015

Ok I'm merging this.

@asfgit asfgit closed this in 208fbca Sep 3, 2015
@vanzin vanzin deleted the SPARK-10421 branch September 9, 2015 23:07
ashangit pushed a commit to ashangit/spark that referenced this pull request Oct 19, 2016
…cies.

This avoids them being mistakenly pulled instead of the newer ones that
Spark actually uses. Spark only depends on these artifacts transitively,
so sometimes maven just decides to pick tachyon's version of the
dependency for whatever reason.

Author: Marcelo Vanzin <[email protected]>

Closes apache#8577 from vanzin/SPARK-10421.

(cherry picked from commit 208fbca)
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.

3 participants