Re: [PATCH] Refactor pqformat.{c,h} and protocol.h - Mailing list pgsql-hackers

From Nathan Bossart
Subject Re: [PATCH] Refactor pqformat.{c,h} and protocol.h
Date
Msg-id ZpboPlg1v38vJhsD@nathan
Whole thread Raw
In response to Re: [PATCH] Refactor pqformat.{c,h} and protocol.h  (Aleksander Alekseev <aleksander@timescale.com>)
Responses Re: [PATCH] Refactor pqformat.{c,h} and protocol.h
List pgsql-hackers
On Tue, Jul 16, 2024 at 10:58:37PM +0300, Aleksander Alekseev wrote:
>> 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?
> 
> Personally I'm fine with either option.

Alright.  Well, I guess I'll flip a coin tomorrow unless someone else
chimes in with an opinion.

-- 
nathan



pgsql-hackers by date:

Previous
From: Nathan Bossart
Date:
Subject: Re: improve performance of pg_dump with many sequences
Next
From: Nazir Bilal Yavuz
Date:
Subject: Re: PG_TEST_EXTRA and meson