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

Поиск
Список
Период
Сортировка
От Michael Paquier
Тема Re: [PATCH] Add additional extended protocol commands to psql: \parse and \bind
Дата
Msg-id ZqG2HAgns3nv3tFY@paquier.xyz
обсуждение исходный текст
Ответ на Re: [PATCH] Add additional extended protocol commands to psql: \parse and \bindx  (Peter Eisentraut <peter@eisentraut.org>)
Ответы Re: [PATCH] Add additional extended protocol commands to psql: \parse and \bind
Список pgsql-hackers
On Wed, Jul 24, 2024 at 05:33:07PM +0200, Peter Eisentraut wrote:
> This commit message confused me, because I don't think this is what the
> \bindx command actually does.  AFAICT, it only binds, it does not execute.
> At least that is what the documentation in the content of the patch appears
> to indicate.

Yep.  FWIW, I always edit these before commit, and noticed that it was
incorrect.  Just took the original message for now.

> I'm not sure \bindx is such a great name.  The "x" stands for "I ran out of
> ideas". ;-)  Maybe \bind_named or \bindn or something like that.  Or use the
> existing \bind with a -name argument?

Not sure that I like much the additional option embedded in the
existing command; I'd rather keep a separate command for each libpq
call, that seems cleaner.  So I would be OK with your suggested
\bind_named.  Fine by me to be outvoted, of course.
--
Michael

Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Michael Paquier
Дата:
Сообщение: Re: XID formatting and SLRU refactorings (was: Add 64-bit XIDs into PostgreSQL 15)
Следующее
От: Thomas Munro
Дата:
Сообщение: Re: CI, macports, darwin version problems