Skip to content

Conversation

@manuzhang
Copy link
Member

@manuzhang manuzhang commented Dec 17, 2024

This is separated from #11750, only to remove Hive runtime.

Copy link
Contributor

@rdblue rdblue left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks @manuzhang!

Copy link
Member

@jbonofre jbonofre left a comment

Choose a reason for hiding this comment

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

LGTM

@manuzhang manuzhang force-pushed the remove-hive-runtime branch from ca3705e to 0c43017 Compare January 2, 2025 03:11
@manuzhang manuzhang force-pushed the remove-hive-runtime branch from 0c43017 to 3b4646f Compare January 10, 2025 13:55
@manuzhang
Copy link
Member Author

The PR has passed voting in the community. @nastra Could you please help to merge?

@manuzhang manuzhang added this to the Iceberg 1.8.0 milestone Jan 10, 2025
@manuzhang manuzhang closed this Jan 13, 2025
@manuzhang manuzhang reopened this Jan 13, 2025
Copy link
Contributor

@Fokko Fokko left a comment

Choose a reason for hiding this comment

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

Thanks for taking care of this @manuzhang and thanks @rdblue, @jbonofre, @nastra and @pan3793 for the review! Let's get this in 🙌

@Fokko Fokko merged commit 7792896 into apache:main Jan 13, 2025
90 of 94 checks passed
@manuzhang manuzhang deleted the remove-hive-runtime branch May 7, 2025 15:46
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.

6 participants