Re: Confusing error message for REINDEX TABLE CONCURRENTLY
David Rowley <david.rowley@2ndquadrant.com>
From: David Rowley <david.rowley@2ndquadrant.com>
To: Ashwin Agrawal <aagrawal@pivotal.io>
Cc: Michael Paquier <michael@paquier.xyz>, PostgreSQL mailing lists <pgsql-hackers@postgresql.org>
Date: 2019-05-28T19:04:46Z
Lists: pgsql-hackers
On Tue, 28 May 2019 at 01:23, Ashwin Agrawal <aagrawal@pivotal.io> wrote: > I think we will need to separate out the NOTICE message for concurrent and regular case. > > For example this doesn't sound correct > WARNING: cannot reindex exclusion constraint index "public.circles_c_excl" concurrently, skipping > NOTICE: table "circles" has no indexes to reindex > > As no indexes can't be reindexed *concurrently* but there are still indexes which can be reindexed, invalid indexes I think fall in same category. Swap "can't" for "can" and, yeah. I think it would be good to make the error messages differ for these two cases. This would serve as a hint to the user that they might have better luck trying without the "concurrently" option. -- David Rowley http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services
Commits
-
Fix confusing NOTICE text in REINDEX CONCURRENTLY
- e24a815c1c85 12.0 landed