| From: | Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com> |
|---|---|
| To: | Etsuro Fujita <etsuro(dot)fujita(at)gmail(dot)com> |
| Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Further cleanup related to statistics import support in postgres_fdw |
| Date: | 2026-08-21 20:40:00 |
| Message-ID: | CALj2ACVrLenZiPrCn1fBST9fAsEwV=VSBKKpfQ1G4DdJZ0j69g@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
On Fri, Aug 21, 2026 at 3:04 AM Etsuro Fujita <etsuro(dot)fujita(at)gmail(dot)com> wrote:
>
> Attached is an updated version of the patch.
The changes look good to me. pgindent wasn't happy about one thing
[1], make check-world is fine (except a known issue in postgres_fdw
being discussed in [2]).
I think this also needs to be backpatched (unless others think
otherwise) because this code is new in PG19 and it's better to keep
the version diff to a minimum.
[1]
dev-dsk-rupiredd-2b-b0ec5a8e % git diff
diff --git a/contrib/postgres_fdw/postgres_fdw.c
b/contrib/postgres_fdw/postgres_fdw.c
index 07a80bf9365..bbadd58d893 100644
--- a/contrib/postgres_fdw/postgres_fdw.c
+++ b/contrib/postgres_fdw/postgres_fdw.c
@@ -393,7 +393,7 @@ typedef struct
{
AttrNumber local_attnum; /* attribute number of local column */
char *local_attname; /* attribute name of local column */
- char *remote_attname; /* attribute name of remote column */
+ char *remote_attname; /* attribute name of remote column */
int res_index; /* index of
row in attribute stats result */
} RemoteAttributeMapping;
--
Bharath Rupireddy
Amazon Web Services: https://aws.amazon.com
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Mario González Troncoso | 2026-08-21 21:19:03 | Re: split tablecmds.c |
| Previous Message | Mario Gonzalez | 2026-08-21 19:56:20 | Re: Possible replace of strncpy on xactdesc.c |