Skip to content
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

Move the description of Iceberg procedures to a dedicated subheading #22845

Merged
merged 1 commit into from
May 29, 2024

Conversation

jackychen718
Copy link
Contributor

According #22608:
Move the description of Iceberg procedures to a dedicated subheading.

Comment on lines 1107 to 1108
=======
>>>>>>> Move the description of Iceberg procedures to a dedicated subheading
Copy link
Member

Choose a reason for hiding this comment

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

Here exists a conflict. Please handle it first.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

conflicts fixed.

@tdcmeehan
Copy link
Contributor

@jackychen718 per our CONTRIBUTING.md, you need to make sure your commits are squashed. Please follow a resource on the internet if you don't know what this means--here is one article I found that may help.

I'll mark this as draft, please take it out of draft once you've completed this, thanks!

@tdcmeehan tdcmeehan marked this pull request as draft May 28, 2024 13:48
Copy link
Contributor

@steveburnett steveburnett left a comment

Choose a reason for hiding this comment

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

Nice work! A few nits and suggestions.

presto-docs/src/main/sphinx/connector/iceberg.rst Outdated Show resolved Hide resolved
presto-docs/src/main/sphinx/connector/iceberg.rst Outdated Show resolved Hide resolved
presto-docs/src/main/sphinx/connector/iceberg.rst Outdated Show resolved Hide resolved
presto-docs/src/main/sphinx/connector/iceberg.rst Outdated Show resolved Hide resolved
presto-docs/src/main/sphinx/connector/iceberg.rst Outdated Show resolved Hide resolved
Copy link
Member

@hantangwangd hantangwangd 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 the fix. Change looks good to me. Please take this PR out of draft following @tdcmeehan 's guidance.

@jackychen718 jackychen718 marked this pull request as ready for review May 29, 2024 13:25
@tdcmeehan
Copy link
Contributor

Looks good to me. @steveburnett would you like to have a look?

Copy link
Contributor

@steveburnett steveburnett left a comment

Choose a reason for hiding this comment

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

LGTM! (docs)

Pull updated branch, reviewed new build of doc, all comments addressed and no new concerns found. Thanks!

@tdcmeehan tdcmeehan self-assigned this May 29, 2024
@tdcmeehan tdcmeehan merged commit 88e5c77 into prestodb:master May 29, 2024
58 checks passed
@wanglinsong wanglinsong mentioned this pull request Jun 25, 2024
36 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[docs] Use a separate chapter or subchapter for describing Iceberg procedures
5 participants