Skip to content

Conversation

@Hisoka-X
Copy link
Member

@Hisoka-X Hisoka-X commented Aug 26, 2024

Purpose of this pull request

Remove hard code iceberg table format version, and clean code.

Does this PR introduce any user-facing change?

no

How was this patch tested?

exist test

Check list

@Hisoka-X Hisoka-X marked this pull request as ready for review August 26, 2024 11:04
@Hisoka-X
Copy link
Member Author

cc @sunxiaojian as well.

Copy link
Member

@wuchunfu wuchunfu left a comment

Choose a reason for hiding this comment

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

LGTM

@hailin0 hailin0 merged commit f49b263 into apache:dev Aug 29, 2024
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.

3 participants