Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SPARK-3955 part 2 [CORE] [HOTFIX] Different versions between jackson-mapper-asl and jackson-core-asl #3829

Closed
wants to merge 1 commit into from

Conversation

srowen
Copy link
Member

@srowen srowen commented Dec 29, 2014

@pwendell 2483c1e didn't actually add a reference to jackson-core-asl as intended, but a second redundant reference to jackson-mapper-asl, as @markhamstra picked up on (#3716 (comment)) This just rectifies the typo. I missed it as well; the original PR #2818 had it correct and I also didn't see the problem.

@SparkQA
Copy link

SparkQA commented Dec 29, 2014

Test build #24864 has started for PR 3829 at commit 6cfdc4e.

  • This patch merges cleanly.

@pwendell
Copy link
Contributor

Gotcha - thanks sean.

@SparkQA
Copy link

SparkQA commented Dec 29, 2014

Test build #24864 has finished for PR 3829 at commit 6cfdc4e.

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

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24864/
Test PASSed.

@jongyoul
Copy link
Member

@srowen Oops... that's my mistake that I didn't change artifact id correctly when I make a pull request again. Thanks for fixing it.

@pwendell
Copy link
Contributor

Thanks Sean - I pulled it in.

@asfgit asfgit closed this in b239ea1 Dec 30, 2014
@srowen srowen deleted the SPARK-3955 branch December 31, 2014 22:20
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.

5 participants