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

From Aleksander Alekseev
Subject Re: [PATCH] Refactor pqformat.{c,h} and protocol.h
Date
Msg-id CAJ7c6TM9W-VYJjiZXZs25YngZkP=fwYSMOrS0LBOZ5hjSmNM6A@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH] Refactor pqformat.{c,h} and protocol.h  (Nathan Bossart <nathandbossart@gmail.com>)
Responses Re: [PATCH] Refactor pqformat.{c,h} and protocol.h
List pgsql-hackers
Hi,

> 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.

-- 
Best regards,
Aleksander Alekseev



pgsql-hackers by date:

Previous
From: Nathan Bossart
Date:
Subject: Re: [PATCH] Refactor pqformat.{c,h} and protocol.h
Next
From: Joe Conway
Date:
Subject: Re: [18] Policy on IMMUTABLE functions and Unicode updates