Re: Request for comment on setting binary format output per session

Поиск
Список
Период
Сортировка
От Dave Cramer
Тема Re: Request for comment on setting binary format output per session
Дата
Msg-id CADK3HH+ZKgNbUpB86fSmKORuHOn7o3zUi9oJanRLJxt6+kiPVA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Request for comment on setting binary format output per session  (Dave Cramer <davecramer@gmail.com>)
Ответы Re: Request for comment on setting binary format output per session  (Daniel Gustafsson <daniel@yesql.se>)
Список pgsql-hackers


On Tue, 25 Apr 2023 at 07:26, Dave Cramer <davecramer@gmail.com> wrote:



On Mon, 24 Apr 2023 at 19:18, Merlin Moncure <mmoncure@gmail.com> wrote:


On Thu, Apr 20, 2023 at 2:52 PM Dave Cramer <davecramer@gmail.com> wrote:

As promised here is a patch with defines for all of the protocol messages.
I created a protocol.h file and put it in src/includes
I'm fairly sure that some of the names I used may need to be changed but the grunt work of finding and replacing everything is done.

In many cases, converting inline character to macro eliminates the need for inline comment, e.g.:
+ case SIMPLE_QUERY: /* simple query */

...that's more work obviously, do you agree and if so would you like some help going through that?

I certainly agree. I left them there mostly for reviewers. I expected some minor adjustments to names of the macro's

So if you have suggestions I'll make changes. 

I'll remove the comments if they are no longer necessary.

Patch attached with comments removed
 

Dave 
Вложения

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

Предыдущее
От: Denis Laxalde
Дата:
Сообщение: Re: Add PQsendSyncMessage() to libpq
Следующее
От: "Daniel Verite"
Дата:
Сообщение: Re: Order changes in PG16 since ICU introduction