Michael Paquier writes:
> Do you think that it would be better to list the letter list for each
> keyword in repl_scanner.l or have something more generic? As not that
> many commands are added in the replication protocol, I would think
> that this is more than enough for each command, say:
> [Ss]
On Sat, May 20, 2017 at 5:32 AM, Tom Lane wrote:
> We'd probably be better off to implement case-insensitivity the hard way.
> There is a reason why none of our other flex scanners use this switch.
Do you think that it would be better to list the letter list for each
keyword in repl_scanner.l or
"Higuchi, Daisuke" writes:
> By adding flex option '-i', replication command parser could be more
> flexible.
> This option is already used for syncrep_scanner.c, so it is not strange to
> add for repl_scanner.c too.
Really? That wasn't an especially bright idea IMO, because it means that
th
On Fri, May 19, 2017 at 12:51 AM, Michael Paquier
wrote:
> On Fri, May 19, 2017 at 1:16 PM, Higuchi, Daisuke
> wrote:
>> The reason of this problem is that sending 'show transaction_read_only' is
>> failed.
>> 'show' must be in uppercase letters because the streaming replication
>> protocol onl
From: Michael Paquier [mailto:michael.paqu...@gmail.com]
> if (!PQsendQuery(conn,
>- "show transaction_read_only"))
>+ "SHOW transaction_read_only"))
>Or perhaps the replication command parser could be made
On Fri, May 19, 2017 at 1:16 PM, Higuchi, Daisuke
wrote:
> The reason of this problem is that sending 'show transaction_read_only' is
> failed.
> 'show' must be in uppercase letters because the streaming replication
> protocol only accepts capital letters.
> In psql and libpq applications that d
Hello,
I found a problem with libpq connection failover.
If primary_conninfo in recovery.conf has 'target_session_attrs=read-write', the
standby fails to start.
How to reproduce the bug:
1. Prepare two standby (standby_1, standby_2) for one master.
On standby_1, primary_conninfo in recove