Repository navigation
test: partition_lifecycle's #1339 findings in both twins: a LIST and two-column drop refusal, an arm not a premise, a 63-byte name - #1361
Merged
jdatcmd merged 3 commits intoOct 14, 2026
Conversation
jdatcmd
approved these changes
Oct 14, 2026
jdatcmd
left a comment
Collaborator
There was a problem hiding this comment.
Approving the content at b8ea210. On PG17 I reproduced the :497 claim with partition_drop_plan's check reduced to IF NOT FOUND:
- main: partition_lifecycle passes 59/59.
- this head: the new arm fails ("a drop plan for a parent not RANGE-partitioned on one column is refused, LIST and two-column RANGE included"): the LIST and two-column parents get through.
The :203 rename is honestly called an accuracy fix, with its off-by-one control.
Needs a rebase before merge. You based this on 25118c0, and #1355 has merged since (main 1933577, 2343 / 1796). With your delta I expect 2344 / 1795.
…: an arm not a premise, a 63-byte name, drop-plan refusals by message Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…lumn parent, in both twins (commandprompt#1339) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Collaborator
Author
|
@jdatcmd, rebased onto ff09c2c (#1359 merged). The new head is a3119fb, replacing b8ea210. Ledger: 2354 / 1797 (2353 + 1; never 1798 − 1).
I'll ping on green CI. |
OffgridwithJD
force-pushed
the
test/1339-partition-lifecycle
branch
from
October 14, 2026 02:13
b8ea210 to
a3119fb
Compare
Collaborator
Author
jdatcmd
approved these changes
Oct 14, 2026
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.
Part of #1339: the
test_partition_lifecyclefindings assigned to me. Each fix is in both twins.Findings
:497[weak], drop planNOT FOUNDrefuses that alone. A new arm adds a LIST parent and a two-column RANGE parent, each led by a timestamptz, so they pass the key check and onlystrat/nattsstop them. Each refusal now asserts its message, not only 55000.IF NOT FOUND: main 59/59 and pytest passed; here "got [55000 …dnl…|ok: 0|ok: 0]", both twins red:92[mislabelled]premise: the parent now has the partitions it reportedchecks the function's own report, so it is an arm (#1236):203[inaccurate]"é"*31 + "x", 63 bytes, as the shell's 63-ASCII-byte name is, and both arms say "63-byte".:77[weak] (missing arms)partition_create_aheadhas an arm.:156was fixed on main before this batch: the boundary rows are computed from the seed, not read from the function's bounds.Evidence
:92arm dated, so the census goes 1798 → 1797;🤖 Generated with Claude Code