| From: | Masahiko Sawada <msawada(at)postgresql(dot)org> |
|---|---|
| To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
| Subject: | pgsql: Unify REPACK (CONCURRENTLY) error messages. |
| Date: | 2026-09-09 20:46:17 |
| Message-ID: | E1x4PBQ-00000004B2R-3dBw@gemulon.postgresql.org |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers |
Unify REPACK (CONCURRENTLY) error messages.
Commit 86f7c82cf102 reported the user catalog table restriction as a
detail, but the other checks in check_concurrent_repack_requirements()
were still a mix: some were hints, and the details repeated
"REPACK (CONCURRENTLY)", which the primary message already names.
Report them as details with the same "This operation ..." wording. The
one actionable hint, ALTER TABLE ... REPLICA IDENTITY USING INDEX,
stays.
Backpatch to v19, where REPACK (CONCURRENTLY) was introduced.
Author: Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com>
Suggested-by: Alvaro Herrera <alvherre(at)kurilemu(dot)de>
Discussion: https://postgr.es/m/apBIFWzHYOaG0auN%40nathan
Backpatch-through: 19
Branch
------
REL_19_STABLE
Details
-------
https://git.postgresql.org/pg/commitdiff/94cc8c443a3115531be7abe414dad5034ee3bb57
Modified Files
--------------
contrib/test_decoding/expected/repack.out | 12 ++++++------
src/backend/commands/repack.c | 22 ++++++++--------------
2 files changed, 14 insertions(+), 20 deletions(-)
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Nathan Bossart | 2026-09-09 21:05:29 | pgsql: Fix hangs in COPY FROM (FORMAT text). |
| Previous Message | Masahiko Sawada | 2026-09-09 20:46:14 | pgsql: Unify REPACK (CONCURRENTLY) error messages. |