-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Docs: Separate page for Branching and Tagging #6723
Docs: Separate page for Branching and Tagging #6723
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
overall LGTM, just a few small things
50791ac
to
3346435
Compare
e1a3651
to
84ff37a
Compare
8f5ae46
to
bc6fa21
Compare
8910681
to
aaa3aee
Compare
4e4207b
to
dcb8cb3
Compare
32fdbd6
to
6ecc6d1
Compare
6ecc6d1
to
5e85dcb
Compare
5e85dcb
to
dcc3126
Compare
There was a problem hiding this 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!
@nastra @hililiwei @namrathamyske @holdenk do you have any further comments? |
dcc3126
to
fec3aa8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks for the work Amogh, and thanks everyone for the review. Since this has been open for quite some time, I will go ahead to merge it, and we can address any additional feedback afterwards. |
This change adds a separate page in Iceberg docs under the "Tables" section focused on branching and tagging.
cc: @jackye1995 @rdblue @namrathamyske @nastra @singhpk234 @rajarshisarkar @aokolnychyi @RussellSpitzer