Skip to content

[Improvement]: Rename Maven property jackson.vesion to jackson.version in mixed Flink modules #4129

@xxubai

Description

@xxubai

Search before asking

  • I have searched in the issues and found no similar issues.

What would you like to be improved?

There is a typo in the mixed Flink Maven property name: jackson.vesion should be jackson.version.

Current references can be found in:

  • amoro-format-mixed/amoro-mixed-flink/pom.xml
  • amoro-format-mixed/amoro-mixed-flink/amoro-mixed-flink-common/pom.xml
  • amoro-format-mixed/amoro-mixed-flink/amoro-mixed-flink-common-iceberg-bridge/pom.xml

This typo does not currently break the build, but it is confusing for contributors and makes the build configuration look inconsistent.

How should we improve?

We should rename the property from jackson.vesion to jackson.version and update all references in the mixed Flink modules.

The change should include:

  • renaming the property in the mixed Flink parent POM
  • updating all dependent module references
  • running a compile or validation command to confirm the renamed property is resolved correctly

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Subtasks

  • Rename jackson.vesion to jackson.version
  • Update all references in mixed Flink module POMs
  • Run a lightweight build validation for the affected modules

Code of Conduct

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions