Horton

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

Dave Cramer <davecramer@gmail.com>

View thread
From: Dave Cramer <davecramer@gmail.com>
To: Merlin Moncure <mmoncure@gmail.com>
Cc: Robert Haas <robertmhaas@gmail.com>, Tom Lane <tgl@sss.pgh.pa.us>, Jeff Davis <pgsql@j-davis.com>, Peter Eisentraut <peter.eisentraut@enterprisedb.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2023-04-25T14:47:19Z
Lists: pgsql-hackers

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

  1. Silence compiler warning introduced in 1edb3b491b

Attachments

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