Re: [HACKERS] pg_control_recovery() return value when not in recovery - Mailing list pgsql-hackers

On 18 September 2017 at 05:50, Andres Freund <andres@anarazel.de> wrote:
> Hi,
>
> Just noticed that we're returning the underlying values for
> pg_control_recovery() without any checks:
> postgres[14388][1]=# SELECT * FROM pg_control_recovery();
>
┌──────────────────────┬───────────────────────────┬──────────────────┬────────────────┬───────────────────────────────┐
> │ min_recovery_end_lsn │ min_recovery_end_timeline │ backup_start_lsn │ backup_end_lsn │
end_of_backup_record_required│
 
>
├──────────────────────┼───────────────────────────┼──────────────────┼────────────────┼───────────────────────────────┤
> │ 0/0                  │                         0 │ 0/0              │ 0/0            │ f
│
 
>
└──────────────────────┴───────────────────────────┴──────────────────┴────────────────┴───────────────────────────────┘
> (1 row)

Yes, that would have made sense for these to be NULL


> postgres[14388][1]=# SELECT pg_is_in_recovery();
> ┌───────────────────┐
> │ pg_is_in_recovery │
> ├───────────────────┤
> │ f                 │
> └───────────────────┘
> (1 row)

But not this, since it is a boolean and the answer is known.

-- 
Simon Riggs                http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

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