On 2016-04-01 20:18:29 +1300, David Rowley wrote:
> On 1 April 2016 at 17:30, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> > David Rowley <david.rowley@2ndquadrant.com> writes:
> >> The attached fixes an error message which is incorrectly using an
> >> unsigned format specifier instead of a signed one.
> >
> > Really though, what
> > astonishes me about this example is that we allow indexes at all on
> > system columns other than OID. None of the other ones can possibly
> > have either a use-case or sensible semantics, can they? We certainly
> > would not stop to update indexes after changing xmax, for example.
>
> As for this part. I really don't see how we could disable this without
> breaking pg_restore for database who have such indexes. My best
> thought is to add some sort of warning during CREATE INDEX, like we do
> for HASH indexes.
As they're currently already not working correctly as indexes, I don't
see throwing an error during pg_restore as being overly harmful.
Andres
