Re: comment doesn't match code - Mailing list pgsql-hackers

From Etsuro Fujita
Subject Re: comment doesn't match code
Date
Msg-id 55791466.5000907@lab.ntt.co.jp
Whole thread Raw
In response to comment doesn't match code  (Robert Haas <robertmhaas@gmail.com>)
List pgsql-hackers
On 2015/06/10 20:18, Robert Haas wrote:
> /*
>   * ALTER TABLE INHERIT
>   *
>   * Add a parent to the child's parents. This verifies that all the columns and
>   * check constraints of the parent appear in the child and that they have the
>   * same data types and expressions.
>   */
> static void
> ATPrepAddInherit(Relation child_rel)
> {
>      if (child_rel->rd_rel->reloftype)
>          ereport(ERROR,
>                  (errcode(ERRCODE_WRONG_OBJECT_TYPE),
>                   errmsg("cannot change inheritance of typed table")));
> }

I agree with you.

Besides that, ISTM taht the error message is a little bit confusing
because I think typed tables cannot inherit.  Maybe I'm missing
something though.  Proposed patch attached.

Best regards,
Etsuro Fujita

Attachment

pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: 9.5 release notes
Next
From: Etsuro Fujita
Date:
Subject: Re: Minor improvement to func.sgml