From: | vignesh C <vignesh21(at)gmail(dot)com> |
---|---|
To: | Ajin Cherian <itsajin(at)gmail(dot)com> |
Cc: | Zheng Li <zhengli10(at)gmail(dot)com>, Peter Smith <smithpb2250(at)gmail(dot)com>, li jie <ggysxcq(at)gmail(dot)com>, Dilip Kumar <dilipbalaut(at)gmail(dot)com>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, "houzj(dot)fnst(at)fujitsu(dot)com" <houzj(dot)fnst(at)fujitsu(dot)com>, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, Japin Li <japinli(at)hotmail(dot)com>, rajesh singarapu <rajesh(dot)rs0541(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Support logical replication of DDLs |
Date: | 2023-01-16 03:12:52 |
Message-ID: | CALDaNm3YEFNwFkkbzS1J8NiHKX6LWQ_h3CA+CPtkUqAZgD8q+Q@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-general pgsql-hackers |
On Sun, 15 Jan 2023 at 09:39, Ajin Cherian <itsajin(at)gmail(dot)com> wrote:
>
> On Fri, Jan 13, 2023 at 5:33 PM vignesh C <vignesh21(at)gmail(dot)com> wrote:
> > Adding support for CREATE/ALTER/DROP Publication ddl deparsing.
> > The attached v61 patch has the changes for the same.
> >
>
> Hi Vignesh,
> this doesn't seem to compile:
>
> gcc -std=gnu99 -Wall -Wmissing-prototypes -Wpointer-arith
> -Wdeclaration-after-statement -Werror=vla -Wendif-labels
> -Wmissing-format-attribute -Wformat-security -fno-strict-aliasing
> -fwrapv -fexcess-precision=standard -g -ggdb -Og -g3
> -fno-omit-frame-pointer -I../../../src/include -D_GNU_SOURCE -c -o
> ddl_deparse.o ddl_deparse.c
> ddl_deparse.c: In function ‘deparse_PublicationObjects’:
> ddl_deparse.c:8956:3: error: unknown type name ‘publication_rel’
> publication_rel *pub_rel = (publication_rel *) lfirst(lc1);
> ^
> ddl_deparse.c:8956:31: error: ‘publication_rel’ undeclared (first use
> in this function)
> publication_rel *pub_rel = (publication_rel *) lfirst(lc1);
> ^
> ddl_deparse.c:8956:31: note: each undeclared identifier is reported
> only once for each function it appears in
> ddl_deparse.c:8956:48: error: expected expression before ‘)’ token
> publication_rel *pub_rel = (publication_rel *) lfirst(lc1);
I was able to apply all the patches and compile successfully, also
cfbot as in [1] has compiled successfully. One thing I noticed is that
structure is available at 006 patch, if you did not apply all the
patch it might fail, we will take care of structure movement in the
next version.
[1] - https://cirrus-ci.com/task/5231672585617408
Regards,
Vignesh
From | Date | Subject | |
---|---|---|---|
Next Message | jian he | 2023-01-16 04:52:10 | Re: does refreshing materialized view make the database bloat? |
Previous Message | pf | 2023-01-15 22:56:41 | Re: pg_upgrade 13.6 to 15.1? [Solved: what can go wrong, will...] |
From | Date | Subject | |
---|---|---|---|
Next Message | David Rowley | 2023-01-16 04:18:32 | Re: Todo: Teach planner to evaluate multiple windows in the optimal order |
Previous Message | Kyotaro Horiguchi | 2023-01-16 03:03:07 | Re: Perform streaming logical transactions by background workers and parallel apply |