Re: BUG #17149: About --skip option of pg_amcheck command

Daniel Gustafsson <daniel@yesql.se>

From: Daniel Gustafsson <daniel@yesql.se>
To: chenjq.jy@fujitsu.com, pgsql-bugs@lists.postgresql.org
Date: 2021-08-17T07:54:12Z
Lists: pgsql-bugs

Attachments

> On 17 Aug 2021, at 06:36, PG Bug reporting form <noreply@postgresql.org> wrote:

>    When I specify the "none" value for --skip option, the following error
> occurs:
>        [postgres14@localhost ~]$ pg_amcheck -p 51403 -d postgres -t tb01
> --skip=none
>        pg_amcheck: error: invalid argument for option --skip
>        [postgres14@localhost ~]$

Nice catch, this should indeed be handled. Fixed in the attached.

>    When I specify the "all-visible" or "all-frozen" value for --skip
> option, take "all-visible" as an example, the following error occurs:
>        [postgres14@localhost ~]$ pg_amcheck -p 51403 -d postgres -t tb01
> --skip=all-visible
>        2021-08-16 04:59:04.571 EDT [4689] ERROR:  invalid skip option
>        2021-08-16 04:59:04.571 EDT [4689] HINT:  Valid skip options are
> "all-visible", "all-frozen", and "none".

Indeed, the attached fixes this as well.

Barring objections I will apply the attached backpatched to 14.

--
Daniel Gustafsson		https://vmware.com/

Commits

  1. Fix pg_amcheck --skip option parameter handling