Repository navigation
Document the Databases and Tables endpoints - #1902
Merged
Merged
Conversation
The databases and tables endpoints, and the DatabaseItem, TableItem and ColumnItem classes, had no API reference entries. Add sections for every method, note that databases.populate_tables is not enabled and show the Metadata API alternative, and list the new explore_databases_tables sample. Closes #735 Co-Authored-By: Claude Code <noreply@anthropic.com>
The link pointed at the delete_dqw_triggers anchor, which is not the method this call makes. Point it at delete_dqws, Delete Data Quality Warning by Content, and correct the link text. Co-Authored-By: Claude Code <noreply@anthropic.com>
populate_tables raises NotImplementedError rather than gating on a placeholder version. delete_table_default_permissions takes (item, rule), and the old order warns. Update requests send only the values that are set, so None leaves a property unchanged and an empty string clears it. Refs #735 Co-Authored-By: Claude Code <noreply@anthropic.com>
content_permissions and description are sent by update, and an empty description clears it. update_tags raises EndpointUnavailableError below REST API 3.9. Co-Authored-By: Claude Code <noreply@anthropic.com>
Revert the two passages that describe unmerged #1901 behavior: - databases.populate_tables is documented as not currently enabled (version 99 gate, EndpointUnavailableError), not as unsupported. - databases.delete_table_default_permissions is documented as (rule, item), not (item, rule), and the tables property row matches the populate_tables wording. Co-Authored-By: Claude Code <noreply@anthropic.com>
databases.update and tables.update send a value only when it is not empty on development, so the update docs say that empty strings do not clear description or certification_note. Revert the "only the values that are set" wording, which depends on unmerged #1901. Co-Authored-By: Claude Code <noreply@anthropic.com>
- DatabaseItem.content_permissions: drop the value list and the claim that databases.update sends it (not true on development; the write and the LockedToDatabase value come with #1901). - Drop the explore_databases_tables.py references (sample is in #1907). - Handle empty results in the databases.populate_tables example. Co-Authored-By: Claude Code <noreply@anthropic.com>
jacalata
added a commit
that referenced
this pull request
Nov 14, 2026
Do not merge before both PRs land: - #1901 (databases and tables fixes): populate_tables raises NotImplementedError, delete_table_default_permissions takes (item, rule), update sends only the values that are set, content_permissions is sent and accepts LockedToDatabase. - #1907 (databases and tables sample): the explore_databases_tables.py references. Also guard the databases.populate_tables example against empty results, matching #1902. Co-Authored-By: Claude Code <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
development:populate_tablesis not currently enabled (version99gate),delete_table_default_permissionstakes(rule, item), and update requests do not send empty values.datasources.delete_dqwREST API link.Refs #735
Related
explore_databases_tables.pysample reference from Add a sample that explores databases and tables #1907) is kept in a separate draft branch,jac/docs-databases-tables-draft-1901. It becomes a PR after both land.DatabaseItem.tagssupport follows in a separate PR.🤖 Generated with Claude Code