Skip to content

Conversation

@rambleraptor
Copy link
Contributor

Rationale for this change

We have some methods for working with tables + namespaces on catalogs that we haven't properly documented. This documentation is super helpful for new users to understand what they can use PyIceberg for. It's also helpful for our LLM...friends? to help understand the project.

Are these changes tested?

Docs only.

Are there any user-facing changes?

Docs only.

Copy link
Contributor

@geruh geruh 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 adding these docs @rambleraptor! Just left a few comments for me

@rambleraptor rambleraptor force-pushed the table_namespace_management_docs branch from 94e5dc2 to db88760 Compare January 27, 2026 18:40
@rambleraptor rambleraptor requested a review from geruh January 27, 2026 18:40
Copy link
Contributor

@kevinjqliu kevinjqliu left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for adding more docs 😄

could you render it locally (make docs-serve) and add screenshots to the pr description?

@rambleraptor
Copy link
Contributor Author

Screenshot 2026-01-27 at 11 27 12 AM Screenshot 2026-01-27 at 11 27 40 AM

@rambleraptor
Copy link
Contributor Author

@kevinjqliu let me know if there's other screenshots you want!

@rambleraptor
Copy link
Contributor Author

Screenshot 2026-01-27 at 12 09 59 PM Screenshot 2026-01-27 at 12 10 25 PM

@rambleraptor
Copy link
Contributor Author

thanks @kevinjqliu @geruh for the review!

@rambleraptor rambleraptor requested a review from geruh January 27, 2026 23:07
Copy link
Contributor

@geruh geruh left a comment

Choose a reason for hiding this comment

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

LGTM!

@kevinjqliu kevinjqliu merged commit a02c0bf into apache:main Jan 27, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants