diff options
| author | David Rowley <drowley@postgresql.org> | 2025-11-07 09:50:02 +1300 |
|---|---|---|
| committer | David Rowley <drowley@postgresql.org> | 2025-11-07 09:50:02 +1300 |
| commit | 448b6a4173d007c75ba30fed666b60f0bd1afe8b (patch) | |
| tree | 1f5e12d536ab095b47ad355f8d3167589a0243e7 /contrib/pg_buffercache/pg_buffercache_pages.c | |
| parent | a2b02293bc65dbb2401cb19c724f52c6ee0f2faf (diff) | |
Tidyup WARNING ereports in subscriptioncmds.c
A couple of ereports were making use of StringInfos as temporary storage
for the portions of the WARNING message. One was doing this to avoid
having 2 separate ereports. This was all fairly unnecessary and
resulted in more code rather than less code.
Refactor out the additional StringInfos and make
check_publications_origin_tables() use 2 ereports.
In passing, adjust pubnames to become a stack-allocated StringInfoData to
avoid having to palloc the temporary StringInfoData. This follows on
from the efforts made in 6d0eba662.
Author: Mats Kindahl <mats.kindahl@gmail.com>
Reviewed-by: David Rowley <dgrowleyml@gmail.com>
Reviewed-by: Amit Kapila <amit.kapila16@gmail.com>
Reviewed-by: Álvaro Herrera <alvherre@kurilemu.de>
Discussion: https://postgr.es/m/0b381b02-cab9-41f9-a900-ad6c8d26c1fc%40gmail.com
Diffstat (limited to 'contrib/pg_buffercache/pg_buffercache_pages.c')
0 files changed, 0 insertions, 0 deletions