Skip to content

NIFI-13576: Upgrade Iceberg from 1.5.2 to 1.6.0#9105

Closed
sullis wants to merge 1 commit intoapache:mainfrom
sullis:NIFI-13576
Closed

NIFI-13576: Upgrade Iceberg from 1.5.2 to 1.6.0#9105
sullis wants to merge 1 commit intoapache:mainfrom
sullis:NIFI-13576

Conversation

@sullis
Copy link
Contributor

@sullis sullis commented Jul 23, 2024

Summary

NIFI-13576

Iceberg 1.6.0 is the latest version
https://github.com/apache/iceberg/releases/tag/apache-iceberg-1.6.0

Issue Tracking

Pull Request Tracking

  • Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
  • Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

  • Pull Request based on current revision of the main branch
  • Pull Request refers to a feature branch with one commit containing changes

Verification

Please indicate the verification steps performed prior to pull request creation.

Build

  • Build completed using mvn clean install -P contrib-check
    • JDK 21

Licensing

  • New dependencies are compatible with the Apache License 2.0 according to the License Policy
  • New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

  • Documentation formatting appears as expected in rendered files

@joewitt
Copy link
Contributor

joewitt commented Jul 23, 2024

@sullis Thanks for keeping an eye on iceberg updates. Have you compared the contents of the resulting nar builds before and after this change? Do those changes require any changes in licensing/notice information? Does it mean we can remove/clean up some of the logic in our poms ?

Iceberg and basically any components from the Hadoop ecosystem require a level of attention/effort that is unique and considerable.

@mattyb149
Copy link
Contributor

I looked at the differences, file sizes for NARs remain the same and there are no new dependencies, just upgrades of existing ones

@mattyb149
Copy link
Contributor

+1 LGTM, thanks for the upgrade! Merging to support/nifi-1.x and main

@mattyb149 mattyb149 closed this in c2dd939 Aug 19, 2024
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