Re: Binary support for pgoutput plugin

Поиск
Список
Период
Сортировка
От Daniel Gustafsson
Тема Re: Binary support for pgoutput plugin
Дата
Msg-id 51CC789A-38D2-4DD9-886A-7BAFDA5C03DE@yesql.se
обсуждение исходный текст
Ответ на Re: Binary support for pgoutput plugin  (Alvaro Herrera <alvherre@2ndquadrant.com>)
Ответы Re: Binary support for pgoutput plugin  (Dave Cramer <davecramer@gmail.com>)
Список pgsql-hackers
> On 5 Jul 2020, at 23:11, Alvaro Herrera <alvherre@2ndquadrant.com> wrote:
>
> On 2020-Jul-05, Daniel Gustafsson wrote:
>
>>> On 2 Jul 2020, at 18:41, Dave Cramer <davecramer@gmail.com> wrote:
>>>
>>> rebased
>>
>> Thanks!  The new version of 0001 patch has a compiler warning due to mixed
>> declarations and code:
>>
>> worker.c: In function ‘slot_store_data’:
>> worker.c:366:5: error: ISO C90 forbids mixed declarations and code [-Werror=declaration-after-statement]
>
> AFAICS this is fixed in 0005.

Yes and no, 0005 fixes one such instance but the one failing the build is
another one in worker.c (the below being from 0008 which in turn change the row
in question from previous patches):

+               int cursor = tupleData->values[remoteattnum]->cursor;

>  I'm going to suggest to use "git rebase -i"

+1

cheers ./daniel


В списке pgsql-hackers по дате отправления:

Предыдущее
От: Alvaro Herrera
Дата:
Сообщение: Re: Binary support for pgoutput plugin
Следующее
От: Tom Lane
Дата:
Сообщение: Re: Can I use extern "C" in an extension so I can use C++?