Skip to content

Conversation

@turboFei
Copy link
Member

@turboFei turboFei commented Nov 14, 2025

Why are the changes needed?

Close #3934.

I met issue with hadoop-3-spark-3.5.

Per the deps list, I saw there are multiple com.fasterxml.jackson versions.

So, I think it is better to manager the jackson libs in dependencyManagement.
image

Brief change log

  • Manage com.fasterxml.jackson dependencies in dependencyManagement to fix conflicts.

How was this patch tested?

  • Add some test cases that check the changes thoroughly including negative and positive cases if possible

  • Add screenshots for manual tests if appropriate
    Testing based on commit: bc5def1

image
  • Run test locally before making a pull request

Documentation

  • Does this pull request introduce a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@turboFei turboFei changed the title Jackson version Manage com.fasterxml.jackson dependencies in dependencyManagement Nov 14, 2025
@turboFei turboFei changed the title Manage com.fasterxml.jackson dependencies in dependencyManagement [AMORO-3934] Manage com.fasterxml.jackson dependencies in dependencyManagement Nov 14, 2025
<hudi.version>0.14.1</hudi.version>
<pagehelper.version>6.1.0</pagehelper.version>
<jsqlparser.version>4.7</jsqlparser.version>
<fasterxml.jackson.version>2.14.2</fasterxml.jackson.version>
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same with

        <amoro-shade-jackson.version>2.14.2</amoro-shade-jackson.version>

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

using fasterxml.jackson.version to clarify with org.codehaus.jackson

@turboFei
Copy link
Member Author

cc @xxubai @zhoujinsong

@turboFei
Copy link
Member Author

gentle ping @zhoujinsong @xxubai

@turboFei
Copy link
Member Author

cc @zhoujinsong @xxubai

@turboFei
Copy link
Member Author

could you help review? @xxubai @zhoujinsong

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: jackson module conficts

1 participant