Fix style in a few REPACK ereports
Álvaro Herrera <alvherre@kurilemu.de>
Fix style in a few REPACK ereports Use consistent "REPACK (CONCURRENTLY)" naming in errhint messages, matching the actual command syntax and the errmsg text used elsewhere in the same file. Also improve the ereport() after XLogReadRecord failure to be like others in the tree. While at it, remove direct mentions of the DDL in the translatable strings, both in the same errhint() calls as well as some errmsg() calls. Add periods where missing. There are all oversights in 28d534e2ae0a. Reported-by: Baji Shaik <baji.pgdev@gmail.com> Discussion: https://postgr.es/m/CA+fm-RPxX1xTcYY4qQGPRDXB2-Fy2SDNdZi=zVjr0j=MPg2PaA@mail.gmail.com
Files
| Path | Change | +/− |
|---|---|---|
| src/backend/commands/repack.c | modified | +10 −5 |
| src/backend/commands/repack_worker.c | modified | +5 −1 |
| src/test/regress/expected/cluster.out | modified | +4 −4 |
Discussion
- [PATCH] Fix errhint messages for REPACK (CONCURRENTLY) restrictions 4 messages · 2026-05-12 → 2026-05-13