Skip to content

[AMORO-4149]: Remove duplicate iceberg-bundled-guava dependency declaration in amoro-format-iceberg.#4150

Merged
xxubai merged 1 commit intoapache:masterfrom
slfan1989:auron-4149
Mar 31, 2026
Merged

[AMORO-4149]: Remove duplicate iceberg-bundled-guava dependency declaration in amoro-format-iceberg.#4150
xxubai merged 1 commit intoapache:masterfrom
slfan1989:auron-4149

Conversation

@slfan1989
Copy link
Copy Markdown
Contributor

Why are the changes needed?

The amoro-format-iceberg/pom.xml file contains duplicate declarations of the iceberg-bundled-guava dependency at two different locations (lines 98-102 and lines 161-167). This redundancy causes unnecessary clutter in the build configuration and could potentially lead to confusion during maintenance.

Close #4149.

Brief change log

  • Remove the duplicate iceberg-bundled-guava dependency declaration (lines 161-167) from amoro-format-iceberg/pom.xml
  • Keep the first declaration (lines 98-102) which is better organized with other iceberg-related dependencies

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

  • 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)

…pendency declaration in amoro-format-iceberg.
@xxubai xxubai changed the title [AMORO-4149] [Improvement]: Remove duplicate iceberg-bundled-guava dependency declaration in amoro-format-iceberg. [AMORO-4149]: Remove duplicate iceberg-bundled-guava dependency declaration in amoro-format-iceberg. Mar 30, 2026
@xxubai xxubai merged commit e9087e3 into apache:master Mar 31, 2026
7 checks passed
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.

[Improvement]: Remove duplicate iceberg-bundled-guava dependency declaration in amoro-format-iceberg

2 participants