Skip to content

docs: correct the partition operations that break publish_via_partition_root - #38432

Open
maheshwarip wants to merge 1 commit into
mainfrom
claude/postgres-partitioned-table-docs-lt7yn8
Open

docs: correct the partition operations that break publish_via_partition_root#38432
maheshwarip wants to merge 1 commit into
mainfrom
claude/postgres-partitioned-table-docs-lt7yn8

Conversation

@maheshwarip

Copy link
Copy Markdown
Contributor

Motivation

Review feedback on #38199: the publish_via_partition_root = true warning on the PostgreSQL partitioned tables page listed adding a partition as one of the operations that surfaces problems. That isn't right. Partitions created with CREATE TABLE ... PARTITION OF are always empty, so they move no data into the root table and cause no divergence. Truncating a partition, which does remove root-table data, was missing from the list.

Description

Replaces "added, attached, or detached" with "attached, detached, or truncated" in the warning. All three of those are DDL that adds or removes root-table data without emitting CDC events, which is precisely the failure mode the warning describes.

Verification

Docs-only prose change; no code or test surface. The paragraph rewraps to stay within the file's line width.


Generated by Claude Code

…on_root

The warning about `publish_via_partition_root = true` listed adding a
partition as one of the operations that surfaces problems. Partitions
created with `CREATE TABLE ... PARTITION OF` are empty, so they move no
data into the root table. Attaching, detaching, and truncating are the
operations that add or remove root-table data without emitting CDC
events, so list those instead.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HBUT5iCf2hkNY2hEfruLTp
@maheshwarip
maheshwarip requested a review from a team as a code owner August 24, 2026 15:10
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.

4 participants