Skip to content

Conversation

@dramaticlly
Copy link
Contributor

Include documentation update for rewrite_data_files procedure for

#9803
#9611
#9724

@manuzhang @himadripal @singhpk234 @szehon-ho if you are interested

@github-actions github-actions bot added the docs label Oct 25, 2024
Copy link
Contributor

@himadripal himadripal left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Member

@szehon-ho szehon-ho left a comment

Choose a reason for hiding this comment

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

Thanks @dramaticlly i left a few comments

Copy link
Member

@szehon-ho szehon-ho left a comment

Choose a reason for hiding this comment

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

Thanks! Just a few more nits

@szehon-ho szehon-ho merged commit 57fb6d5 into apache:main Oct 31, 2024
@szehon-ho
Copy link
Member

Merged, thanks @dramaticlly , also @RussellSpitzer @himadripal @huaxingao @singhpk234 for reviews !

@dramaticlly dramaticlly deleted the rewriteDoc branch December 12, 2024 19:04
zachdisc pushed a commit to zachdisc/iceberg that referenced this pull request Dec 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants