On 2017-01-19 19:12, Petr Jelinek wrote:
On 19/01/17 18:44, Erik Rijkers wrote:


Could probably be whittled down to something shorter but I hope it's
still easily reproduced.


Just analyze on the pg_subscription is enough.


heh. Ah well, I did find it :)


Can you give the current patch set? I am failing to get a compilable set.

In the following order they apply, but then fail during compile.

0001-Add-PUBLICATION-catalogs-and-DDL-v18.patch
0002-Add-SUBSCRIPTION-catalog-and-DDL-v18.patch
0003-Define-logical-replication-protocol-and-output-plugi-v18.patch
0004-Add-logical-replication-workers-v18fixed.patch
0006-Add-RENAME-support-for-PUBLICATIONs-and-SUBSCRIPTION.patch
0001-Logical-replication-support-for-initial-data-copy-v3.patch
pg_subscription-analyze-fix.diff

The compile fails with:

In file included from ../../../../src/include/postgres.h:47:0,
                 from worker.c:27:
worker.c: In function ‘create_estate_for_relation’:
../../../../src/include/c.h:203:14: warning: passing argument 4 of ‘InitResultRelInfo’ makes pointer from integer without a cast [-Wint-conversion]
 #define true ((bool) 1)
              ^
worker.c:187:53: note: in expansion of macro ‘true’
  InitResultRelInfo(resultRelInfo, rel->localrel, 1, true, NULL, 0);
                                                     ^~~~
In file included from ../../../../src/include/funcapi.h:21:0,
                 from worker.c:31:
../../../../src/include/executor/executor.h:189:13: note: expected ‘Relation {aka struct RelationData *}’ but argument is of type ‘char’
 extern void InitResultRelInfo(ResultRelInfo *resultRelInfo,
             ^~~~~~~~~~~~~~~~~
worker.c:187:59: warning: passing argument 5 of ‘InitResultRelInfo’ makes integer from pointer without a cast [-Wint-conversion]
  InitResultRelInfo(resultRelInfo, rel->localrel, 1, true, NULL, 0);
                                                           ^~~~
In file included from ../../../../src/include/funcapi.h:21:0,
                 from worker.c:31:
../../../../src/include/executor/executor.h:189:13: note: expected ‘int’ but argument is of type ‘void *’
 extern void InitResultRelInfo(ResultRelInfo *resultRelInfo,
             ^~~~~~~~~~~~~~~~~
worker.c:187:2: error: too many arguments to function ‘InitResultRelInfo’
  InitResultRelInfo(resultRelInfo, rel->localrel, 1, true, NULL, 0);
  ^~~~~~~~~~~~~~~~~
In file included from ../../../../src/include/funcapi.h:21:0,
                 from worker.c:31:
../../../../src/include/executor/executor.h:189:13: note: declared here
 extern void InitResultRelInfo(ResultRelInfo *resultRelInfo,
             ^~~~~~~~~~~~~~~~~
make[4]: *** [worker.o] Error 1
make[4]: *** Waiting for unfinished jobs....
make[3]: *** [logical-recursive] Error 2
make[2]: *** [replication-recursive] Error 2
make[2]: *** Waiting for unfinished jobs....
^[make[1]: *** [all-backend-recurse] Error 2
make: *** [all-src-recurse] Error 2




but perhaps that patchset itself is incorrect, or the order in which I applied them.

Can you please put them in the right order?  (I tried already a few...)


thanks,


Erik Rijkers



--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to