Обсуждение: [HACKERS] Making replication commands case-insensitive

Поиск
Список
Период
Сортировка

[HACKERS] Making replication commands case-insensitive

От
Michael Paquier
Дата:
Hi all,

$subject has been raised in a recent thread here:
https://www.postgresql.org/message-id/CAB7nPqTmym5t-X6hvMF_P-KRc=ndXtbQCTiU=NhS_jvL7x1kCA@mail.gmail.com

The idea is to make the replication protocol a bit more flexible, in a
way similar to what 5c837dd has done, but for repl_scanner.l. This
will also allow to avoid any problems like what has been fixed in
aa41bc7 where the SHOW commands used in libpq have to be capitalized.
Personally, I have pested about the lack of flexibility a couple of
times when running tests using psql..

I am parking that in the next CF.
Thanks,
-- 
Michael

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Вложения

Re: [HACKERS] Making replication commands case-insensitive

От
Magnus Hagander
Дата:


On Sat, May 20, 2017 at 2:19 PM, Michael Paquier <michael.paquier@gmail.com> wrote:
Hi all,

$subject has been raised in a recent thread here:
https://www.postgresql.org/message-id/CAB7nPqTmym5t-X6hvMF_P-KRc=ndXtbQCTiU=NhS_jvL7x1kCA@mail.gmail.com

The idea is to make the replication protocol a bit more flexible, in a
way similar to what 5c837dd has done, but for repl_scanner.l. This
will also allow to avoid any problems like what has been fixed in
aa41bc7 where the SHOW commands used in libpq have to be capitalized.
Personally, I have pested about the lack of flexibility a couple of
times when running tests using psql..

I am parking that in the next CF.

Given that the protocol really isn't intended for "manual consumption", do we really want that? Not that it adds a lot of complexity, but still. It certainly makes the code completely unreadable. And since any program using it should figure out pretty quickly that it's not working if they us the wrong casing...

--

Re: [HACKERS] Making replication commands case-insensitive

От
Andres Freund
Дата:
Hi,

On 2017-05-20 21:19:10 +0900, Michael Paquier wrote:
> $subject has been raised in a recent thread here:
> https://www.postgresql.org/message-id/CAB7nPqTmym5t-X6hvMF_P-KRc=ndXtbQCTiU=NhS_jvL7x1kCA@mail.gmail.com
> 
> The idea is to make the replication protocol a bit more flexible, in a
> way similar to what 5c837dd has done, but for repl_scanner.l. This
> will also allow to avoid any problems like what has been fixed in
> aa41bc7 where the SHOW commands used in libpq have to be capitalized.
> Personally, I have pested about the lack of flexibility a couple of
> times when running tests using psql..

-1.  This solution is too bad for maintainability, and given the
replication protocol is not for interactive use I don't see a
corresponding benefit.

Greetings,

Andres Freund