Re: [PATCH] Refactor pqformat.{c,h} and protocol.h
От | Nathan Bossart |
---|---|
Тема | Re: [PATCH] Refactor pqformat.{c,h} and protocol.h |
Дата | |
Msg-id | ZpbOknZUTGZ8G5Vx@nathan обсуждение исходный текст |
Ответ на | Re: [PATCH] Refactor pqformat.{c,h} and protocol.h (Aleksander Alekseev <aleksander@timescale.com>) |
Ответы |
Re: [PATCH] Refactor pqformat.{c,h} and protocol.h
|
Список | pgsql-hackers |
On Tue, Jul 16, 2024 at 09:14:35PM +0300, Aleksander Alekseev wrote: >> As discussed elsewhere [0], we can add the leader/worker protocol >> characters to protocol.h, but they should probably go in a separate >> section. I'd recommend breaking that part out to a separate patch, too. > > OK, here is the updated patchset. This time I chose not to include this patch: Thanks. The only thing that stands out to me is the name of the parallel leader/worker protocol message. In the original thread for protocol characters, some early versions of the patch called it a "parallel progress" message, but this new one just calls it PqMsg_Progress. I guess PqMsg_ParallelProgress might be a tad more descriptive and less likely to cause naming collisions with new frontend/backend messages, but I'm not tremendously worried about either of those things. Thoughts? -- nathan
В списке pgsql-hackers по дате отправления: