Re: [PATCH] Add additional extended protocol commands to psql: \parse and \bindx

On 7/19/24 04:23, Michael Paquier wrote:
> On Fri, Jul 19, 2024 at 12:17:44AM +0200, Tomas Vondra wrote:
>> shall we do something about this patch? It seems to be in a pretty good
>> shape (pretty much RFC, based on quick review), the cfbot is still
>> happy, and there seems to be agreement this is a nice feature.
>>
>> Michael, I see you've reviewed the patch in January. Do you agree / plan
>> to get it committed, or should I take a look?
>
> This feel off my radar a bit, thanks for the reminder :)
>
> I have a local branch dating back from January where this patch is
> sitting, with something like 50% of the code reviewed. I'd still need
> to look at the test coverage, but I did like the proposed patch a lot
> based on my notes.
>
> I may be able to come back to that if not next week, then the week
> after that. If you want to handle it yourself before that, that's
> fine by me.

OK, if you're already half-way through the review, I'll leave it up to
you. I don't think we need to rush, and I'd have to learn about all the
psql stuff first anyway.

regards

--
Tomas Vondra
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company